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

feat: optimize deployment and tally event #1083

Merged
merged 1 commit into from
Jan 23, 2024
Merged

feat: optimize deployment and tally event #1083

merged 1 commit into from
Jan 23, 2024

Conversation

0xmad
Copy link
Collaborator

@0xmad 0xmad commented Jan 22, 2024

Description

  • Update integration and cli docs
  • Add poseidon contract addresses as cli arguments
  • Add BallotsTallied(address) event for Tally contract
  • Reuse already deployed poseidon contracts
  • Use network to distingish contracts for different deployments

Additional Notes

Storage layout (contracts) and file with contract addresses (cli) are changed

Related issue(s)

Closes #1054
Closes #1060
Closes #937

Confirmation

@0xmad 0xmad added documentation Improvements or additions to documentation chore Chore tasks labels Jan 22, 2024
@0xmad 0xmad self-assigned this Jan 22, 2024
Copy link

netlify bot commented Jan 22, 2024

Deploy Preview for maci-typedoc ready!

Name Link
🔨 Latest commit 185b407
🔍 Latest deploy log https://app.netlify.com/sites/maci-typedoc/deploys/65afe1f9e89cb60008415dae
😎 Deploy Preview https://deploy-preview-1083--maci-typedoc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@0xmad 0xmad force-pushed the feature/poll-event branch from eea66aa to 5be0b71 Compare January 23, 2024 00:14
@0xmad 0xmad changed the title refactor: optimize deployment and tally event feat: optimize deployment and tally event Jan 23, 2024
@0xmad 0xmad added the enhancement New feature or request label Jan 23, 2024
@0xmad 0xmad marked this pull request as ready for review January 23, 2024 00:19
Copy link
Collaborator

@ctrlc03 ctrlc03 left a comment

Choose a reason for hiding this comment

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

Great work, very neat. Just left one suggestion

contracts/contracts/Tally.sol Outdated Show resolved Hide resolved
Copy link
Collaborator

@kittybest kittybest left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@ctrlc03 ctrlc03 force-pushed the feature/poll-event branch from e5223f4 to da94737 Compare January 23, 2024 14:45
- [x] Update integration and cli docs
- [x] Add poseidon contract addresses as cli arguments
- [x] Add BallotsTallied(address) event for Tally contract
- [x] Reuse already deployed poseidon contracts
- [x] Use network to distingish contracts for different deployments
@0xmad 0xmad force-pushed the feature/poll-event branch from da94737 to 185b407 Compare January 23, 2024 15:57
@0xmad 0xmad requested a review from ctrlc03 January 23, 2024 15:58
Copy link
Collaborator

@ctrlc03 ctrlc03 left a comment

Choose a reason for hiding this comment

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

LGTM thank you!!

@ctrlc03 ctrlc03 merged commit 5b736d9 into dev Jan 23, 2024
17 of 18 checks passed
@ctrlc03 ctrlc03 deleted the feature/poll-event branch January 23, 2024 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore tasks documentation Improvements or additions to documentation enhancement New feature or request
Projects
Status: Done
3 participants