Raise exception and fix multi-word issue #59

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.

Sign up for GitHub

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

Jump to bottom

Open

123vivekr wants to merge 2 commits into geekpradd:master from 123vivekr:fix

123vivekr commented Sep 10, 2022

This PR fixes two things:

Adds support for multi-word words in `meaning API
Raises WordNotFound exception if dictionary API throws error
meaning API raises exception if dictionary API throws error

Note: This supports only the meanings API

Fixes #55

123vivekr added 2 commits

September 10, 2022 14:34


          meaning: support multiple word input

a0a288d


          throw error on API fetch failure

506ddf6

123vivekr force-pushed the fix branch from a102bd1 to 506ddf6 Compare

September 10, 2022 09:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet