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

Wagmi prepare transaction for votes #668

Closed
Tracked by #669
jaybuidl opened this issue Apr 5, 2023 · 1 comment · Fixed by #670
Closed
Tracked by #669

Wagmi prepare transaction for votes #668

jaybuidl opened this issue Apr 5, 2023 · 1 comment · Fixed by #670

Comments

@jaybuidl
Copy link
Member

jaybuidl commented Apr 5, 2023

Follow-up on #624

@alcercu
Copy link
Contributor

alcercu commented Apr 5, 2023

Just to add more context: wagmi uses hooks to prepare transactions. Due to the rules of hooks there are certain limitations.

In our current design, we send the transaction when the user clicks the option they want to vote for, making it impossible to preemptively run the hook (as the option is not known until the user clicks). We would need to independently select voting option, upload justification and send transaction, as the hook can then run after the option and the justification CID is known and before they click send vote.

@jaybuidl jaybuidl added the Package: Web Court web frontend label Apr 10, 2023
@jaybuidl jaybuidl linked a pull request Jun 19, 2023 that will close this issue
@jaybuidl jaybuidl added this to the alpha-1 milestone Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants