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

[MODORDERS-1232]. SPIKE - Investigate whether javamoney can be used with alternate exchange rate sources #1087

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BKadirkhodjaev
Copy link
Contributor

@BKadirkhodjaev BKadirkhodjaev commented Jan 22, 2025

Purpose

Approach

  • This PR is not for merging but is rather a proof-of-concept that Java Money Moneta can support the inclusion of a custom exchange rate API

@BKadirkhodjaev BKadirkhodjaev requested a review from a team January 22, 2025 08:29
@BKadirkhodjaev BKadirkhodjaev changed the title [MODORDERS-1232]. SPIKE - Investigate whether javamoney can be used w… [MODORDERS-1232]. SPIKE - Investigate whether javamoney can be used with alternate exchange rate sources Jan 22, 2025
@BKadirkhodjaev BKadirkhodjaev added the DO NOT MERGE External dependencies. Do not merge yet. label Jan 22, 2025
private static final String GET_LATEST_EXCHANGE_RATE_URL = "https://api.currencyapi.com/v3/latest?apikey=%s&base_currency=%s&currencies=%s";
private static final String CONTENT_TYPE = "Content-Type";
private static final String APPLICATION_JSON_UTF_8 = "text/application-json;charset=UTF-8";
private static final String apiKey = System.getenv("API_KEY");
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An API_KEY must be supplied for it to work (create a trial account here: https://currencyapi.com)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE External dependencies. Do not merge yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant