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: sunset NEAR conversion for now #1302

Merged
merged 6 commits into from
Dec 19, 2023
Merged

Conversation

yomarion
Copy link
Member

@yomarion yomarion commented Dec 18, 2023

Description of the changes

The previous oracle we implemented is down. We should investigate supporting Pyth network instead.

Removed:

  • references to Flux oracles
  • Near pairs from default conversion pairs (=> this is debatable, see More details)

Attempts to pay a Near request with conversion throws with Payments with on-chain conversions are currently disabled on Near.

More details

Removing Near pairs is debatable, it's quite likely that we will add them back soon. If we think of this array as a stable default set on top of which implementers add their implementations, it's a poor choice and it's up to implementers to remove these values from the default values.


@coveralls
Copy link

coveralls commented Dec 18, 2023

Coverage Status

coverage: 87.093% (-0.01%) from 87.105%
when pulling 8ed4cb0 on fix/sunset-NEAR-conversion
into fcbf982 on master.

Copy link
Member

@MantisClone MantisClone left a comment

Choose a reason for hiding this comment

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

I'm okay with removing the fluxCurrencyPairs from the default list for now. 👍

@yomarion yomarion enabled auto-merge (squash) December 19, 2023 13:44
@yomarion yomarion merged commit afdb449 into master Dec 19, 2023
25 checks passed
@yomarion yomarion deleted the fix/sunset-NEAR-conversion branch December 19, 2023 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants