Skip to content

Conversation

@cctdaniel
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Jan 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
xc-admin-frontend ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 25, 2023 at 1:39AM (UTC)

@cctdaniel cctdaniel requested a review from guibescos January 24, 2023 06:37
@cctdaniel cctdaniel marked this pull request as ready for review January 24, 2023 06:37

return () => {}
return () => {
cancelled = true
Copy link
Contributor Author

@cctdaniel cctdaniel Jan 24, 2023

Choose a reason for hiding this comment

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

without this line, and the added lines above, it broke the site because it was returning multiple cluster data and the order wasn't preserved

Copy link
Contributor

Choose a reason for hiding this comment

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

Me not understand but I'll trust you

)!.address

pythProgramClient.methods
.setMinPub(newMinPublishers, [0, 0, 0])
Copy link
Contributor

@guibescos guibescos Jan 24, 2023

Choose a reason for hiding this comment

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

In accounts, you need to make fundingAccount : squads?.getAuthorityPDA( SECURITY_MULTISIG[getMultisigCluster(cluster)], 1 ) otherwise the multisig won't sign the transaction

Copy link
Contributor

@guibescos guibescos left a comment

Choose a reason for hiding this comment

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

Need to fix :

  • proposal accounts
  • video that I sent you

Copy link
Contributor

@guibescos guibescos left a comment

Choose a reason for hiding this comment

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

🚢 it sir

@cctdaniel cctdaniel merged commit 2c6eb7d into main Jan 25, 2023
@cctdaniel cctdaniel deleted the min-pub branch January 25, 2023 04:50
ali-behjati pushed a commit that referenced this pull request Jan 25, 2023
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