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

Replace NFT with denoms #186

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

Replace NFT with denoms #186

wants to merge 9 commits into from

Conversation

deniszagumennov
Copy link
Contributor

No description provided.

@deniszagumennov deniszagumennov force-pushed the feat/withdrawal-token branch 3 times, most recently from 98e23cc to b2d6808 Compare September 30, 2024 09:50
@deniszagumennov deniszagumennov changed the title Initial commit Replace NFT with denims Oct 2, 2024
@deniszagumennov deniszagumennov changed the title Replace NFT with denims Replace NFT with denoms Oct 2, 2024
@deniszagumennov deniszagumennov force-pushed the feat/withdrawal-token branch 3 times, most recently from 81d321f to 09d7218 Compare October 3, 2024 11:07
@deniszagumennov deniszagumennov force-pushed the feat/withdrawal-token branch 2 times, most recently from f9dc653 to 76b520a Compare October 9, 2024 17:03
@deniszagumennov deniszagumennov force-pushed the feat/withdrawal-token branch 2 times, most recently from 3e395b3 to f460932 Compare October 14, 2024 14:33
@deniszagumennov deniszagumennov force-pushed the feat/withdrawal-token branch 2 times, most recently from b02d306 to df38ce3 Compare October 16, 2024 11:51
Copy link
Member

@faust403 faust403 left a comment

Choose a reason for hiding this comment

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

I think we forgot to add new contracts into scripts/deploy_scripts/upload_contracts.bash

@foxpy
Copy link
Contributor

foxpy commented Oct 17, 2024

I think overall you are going in a correct direction, I didn't find any major flaws in your code.

Please take a careful look at a bunch of suggestions and comments I prepared here: #190
Keep in mind, even though I have only patched auto-withdrawer, some of the patches (most notably avoiding excess copying) are applicable to all contracts.

P.S. I thought it would be more convenient to write some code instead of using the Github's review system

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