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

fix: change currency exchange API to api.frankfurter.app #37355

Merged

Conversation

dsnetprofitxl
Copy link

api.exchangerate.host now requires an API key to use.

switching to api.frankfurter.app

@dsnetprofitxl dsnetprofitxl marked this pull request as ready for review October 4, 2023 00:57
@ruthra-kumar ruthra-kumar self-assigned this Oct 4, 2023
@ruthra-kumar
Copy link
Member

Test case updated to use exchange rate from frankfurter.

https://api.frankfurter.app/2015-12-15?from=USD&to=INR
Screenshot 2023-10-04 at 9 01 20 PM

https://api.frankfurter.app/2016-01-15?from=USD&to=INR
Screenshot 2023-10-04 at 9 01 22 PM

@ruthra-kumar ruthra-kumar merged commit c86cd99 into frappe:version-13-hotfix Oct 4, 2023
6 of 7 checks passed
@ruthra-kumar
Copy link
Member

@Mergifyio backport version-13

@mergify
Copy link
Contributor

mergify bot commented Oct 5, 2023

backport version-13

✅ Backports have been created

ruthra-kumar added a commit that referenced this pull request Oct 5, 2023
fix: change currency exchange API to api.frankfurter.app (backport #37355)
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 13.54.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@wojosc
Copy link
Contributor

wojosc commented Oct 10, 2023

would it not make sense to place this URL to e.g. "Accounts Settings" to have the user chose from different providers if necessary any maybe only set a standard or fallback via hard-code?

@ruthra-kumar
Copy link
Member

@wojosc
That provision was subsequently added in #37385 -- v13.54.2

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants