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

[Master] Add weighted allowlist: 80REG-20WXDAI #5641

Merged
merged 1 commit into from
May 2, 2024

Conversation

0xEzSwim
Copy link
Contributor

@0xEzSwim 0xEzSwim commented May 2, 2024

Description

Added the following pool ID to the Weighted allowlist:

  • 80REG-20WXDAI : 0x915938ff0a2b54e575068ff1a95007cafba7d27700020000000000000000008a

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Dependency changes
  • Code refactor / cleanup
  • Documentation or wording changes
  • Other

How should this be tested?

Check Weighted allowlist 80REG-20WXDAI pool ID == 0x915938ff0a2b54e575068ff1a95007cafba7d27700020000000000000000008a

Visual context

No UI changes

Checklist:

  • I have performed a self-review of my own code
  • I have requested at least 1 review (If the PR is significant enough, use best judgement here)
  • I have commented my code where relevant, particularly in hard-to-understand areas
  • If package-lock.json has changes, it was intentional.
  • The base of this PR is master if hotfix, develop if not

Copy link

vercel bot commented May 2, 2024

@0xEzSwim is attempting to deploy a commit to the Balancer Team on Vercel.

A member of the Team first needs to authorize it.

@0xEzSwim 0xEzSwim changed the base branch from develop to master May 2, 2024 08:28
@0xEzSwim
Copy link
Contributor Author

0xEzSwim commented May 2, 2024

@garethfuller could you please review the pool I want to add to the gnosis weighted allowlist pretty please :)

@0xEzSwim 0xEzSwim changed the title Master - Weighted.Allowlist 80REG-20WXDAI [Master] Add weighted allowlist: 80REG-20WXDAI May 2, 2024
@Tritium-VLK Tritium-VLK self-requested a review May 2, 2024 08:54
Copy link

vercel bot commented May 2, 2024

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
app-v2 ⬜️ Ignored (Inspect) Visit Preview May 2, 2024 9:00am

@agualis agualis merged commit 9c7e4f1 into balancer:master May 2, 2024
5 of 6 checks passed
@agualis
Copy link
Collaborator

agualis commented May 2, 2024

Pool allowlisted ✅
Thanks for the PR @0xEzSwim

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.

2 participants