-
Notifications
You must be signed in to change notification settings - Fork 39
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
Parax nft #405
Parax nft #405
Conversation
helpers/contracts-deployments.ts
Outdated
verify | ||
); | ||
|
||
const proxyAdmin = "0x17816E9A858b161c3E37016D139cf618056CaCD4"; //yubo.eth |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems hard coded
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might better be our safe address? 0x19293FBec52F94165f903708a74513Dd6dFedd0a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a one-time deploy contract and yubo ask use his addrss as proxy admin
test/_parax_spec.ts
Outdated
expect(await ParaX.totalSupply()).to.be.eq(4); | ||
|
||
expect(await getProxyAdmin(ParaX.address)).to.be.eq( | ||
"0x17816E9A858b161c3E37016D139cf618056CaCD4" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
Security Checklist
Make sure to think about each of these exploits in this PR.