Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat (wolfram): added wolfram command #66

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

BenSegal855
Copy link
Contributor

Describe this PR and why it should be merged:
This PR adds the wolfram command so Steve can do calculus. This does require a wolfram app id which can be gotten here. Im also using .then() syntax because I dont want to deal with errors twice and axios will throw an error if the server responds with a 500 level error code. Only other change was that I added the NASA value to the config example because it was missing.
Status

  • This PR has been tested and complies with this project's ESLint rules
  • This PR adds/modifes a command

Semantic Versioning

  • This PR features bug fixes, or only style changes/refactorings, or no code changes
  • This PR adds features for users
  • This PR switches this project to a new a new Discord API library/new Discord.js framework

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant