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

#797 - DCV_J should be in OCEAN #798

Merged

Conversation

trizin
Copy link
Contributor

@trizin trizin commented Feb 6, 2024

Fixes #797

Changes proposed in this PR:

  • Update DCV used in capping to be in OCEAN
  • Update tests

@trizin trizin linked an issue Feb 6, 2024 that may be closed by this pull request

stakes = {C1: {NA: {LP1: 1e6}}}
nftvols = {C1: {OCN_ADDR: {NA: DCV_USD}}}
nftvols = {C1: {OCN_ADDR: {NA: DCV_OCEAN}}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: Reward calculator expects volumes to be in the token used to purchase the asset and later converts it to USD.

Copy link

codeclimate bot commented Feb 6, 2024

Code Climate has analyzed commit 5951384 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 92.7% (0.0% change).

View more on Code Climate.

@trizin trizin requested a review from trentmc February 6, 2024 11:34
@trizin
Copy link
Contributor Author

trizin commented Feb 6, 2024

Please ignore the failing "Dispense Active Rewards" failure, it is failing because "https://safe-transaction-goerli.safe.global" is down.

@trizin trizin marked this pull request as ready for review February 6, 2024 15:09
@trizin trizin merged commit 70119d3 into main Feb 6, 2024
6 of 7 checks passed
@trizin trizin deleted the issue797-dcv_j-is-in-usd-and-being-used-to-limit-the-max-ocean branch February 6, 2024 15:22
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.

DCV_j is in USD and being used to limit the max OCEAN
2 participants