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

[Feature] Obtain the Exchange Rate field from somewhere else other than the receipt field in Transactions #71

Open
1 of 4 tasks
fivetran-joemarkiewicz opened this issue Dec 18, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@fivetran-joemarkiewicz
Copy link
Contributor

Is there an existing feature request for this?

  • I have searched the existing issues

Describe the Feature

Currently, the package obtains the exchange_rate field from the Transactions object in a JSON field titled receipt. This works to obtain the exchange rate provided within the receipt. However, the receipt for different payment processors is not always the same and also may not be entirely accurate (especially when considering returns).

This FR is to leverage some other method of obtaining the exchange_rate field to be used within the downstream transformation logic in this package.

Describe alternatives you've considered

At the moment we will continue to leverage the receipt field in the Transactions table.

Unfortunately, we are unaware if there is another solution. But this FR is intended for the team and community to ideally find an alternative for the package to leverage.

Are you interested in contributing this feature?

  • Yes.
  • Yes, but I will need assistance and will schedule time during your office hours for guidance.
  • No.

Anything else?

No response

@fivetran-joemarkiewicz fivetran-joemarkiewicz added the enhancement New feature or request label Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant