OriginProtocol / OriginProtocol/origin-dollar
Update DApp and Analytics when upgrading Vault for OUSD
Nobody has claimed this yet.
- Dominant language
- Solidity
- Stars
- 152
- Forks
- 115
- Avg merge
- 3d 20h
- Merged PRs (30d)
- 17
Description
The OETH PR #1271 renames the functions priceUSDMint and priceUSDRedeem to priceUnitMint and priceUnitRedeem respectively. But since we haven't upgraded the OUSD Vault yet, we have to be mindful to update the code of DApp and Analytics to use the renamed functions when we finally get to upgrade OUSD Vault.
Doing this will make sure that we don't accidentally break DApp or the Analytics part when the upgrade happens.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating references to priceUSDMint and priceUSDRedeem in the DApp and Analytics code. Check how those references interact with the OUSD Vault upgrade, then update them to priceUnitMint and priceUnitRedeem. Done means both integrations use the renamed functions without breaking when the OUSD Vault is upgraded.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- solidity
- Domain
- blockchain
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100