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

Add MATICX as collateral in Polygon USDC market #780

Merged
merged 8 commits into from
Aug 24, 2023

Conversation

kevincheng96
Copy link
Contributor

@kevincheng96 kevincheng96 commented Jul 13, 2023

MATICX team's proposal to add MATICX as a collateral asset on the Polygon cUSDCv3 market.

@kevincheng96
Copy link
Contributor Author

Cleaned up this PR a bit.

Manoj Patra and others added 4 commits July 19, 2023 15:08
* initial migration script to add maticx as collateral for polygon usdc

* add verify block

* formatting: white spaces removal
* minor fix

* linting fix
* maticx yaml permissions fix

* Upgrade git action to use new seacrest (#782)

* upgrade git action to use new seacrest

* updated asset config values

* added description
@kevincheng96 kevincheng96 force-pushed the manoj9april/add-maticx branch from f03f678 to 9519292 Compare July 19, 2023 22:08
@kevincheng96 kevincheng96 force-pushed the manoj9april/add-maticx branch from 9519292 to 0050f7a Compare July 19, 2023 23:47
@manoj9april
Copy link

manoj9april commented Jul 24, 2023

@kevincheng96 @scott-silver please review this #796

* code refactor

* lint fix

* removed unwanted code
@manoj9april
Copy link

@kevincheng96 Please review this as well #798
contains only description changes

@kevincheng96
Copy link
Contributor Author

This proposal was executed: https://compound.finance/governance/proposals/172

@kevincheng96 kevincheng96 merged commit 658f451 into main Aug 24, 2023
@kevincheng96 kevincheng96 deleted the manoj9april/add-maticx branch August 24, 2023 00:24
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.

3 participants