-
Notifications
You must be signed in to change notification settings - Fork 81
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
better OpenExchangeRates error reporting #38
Comments
Should now be fixed on my fork which has completely restructured the management commands to integrate other sources. Yahoo Finance now integrated. Check it out! I am just going to raise a pull request :-) |
@racitup +1 for PR |
@racitup @bashu I wonder if you might be better off just moving the actual openexchangerates module into this project. It seems to be abandoned and do not support python 3 because of this tiny bug that has a PR waiting for almost a year metglobal/openexchangerates#6 openexchangerates has an MIT license so that should not be a problem. |
Show me the pull requests :) |
See PR #43 :) |
Changing the base currency from USD to some other currency requires a paid account with OpenExchangeRates.
Currently the API returns an error that doesn't make this clear at first glance:
Copying the request URL to another tab reveals the problem but this exception should be handled appropriately.
The text was updated successfully, but these errors were encountered: