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(permit): remove permit related feature flags #3320

Merged
merged 2 commits into from
Nov 3, 2023

Conversation

alfetopito
Copy link
Collaborator

Summary

Remove permit related feature flags

Permit is already enabled in prod, no longer need the feature flags.
Even for gnosis chain - which is currently not enabled - there's at least one token identified as permittable
So it should be safe to enable it there as well.
Further work is still needed to fix it for other tokens, but it should be fine.

To Test

  1. Permit should work as usual

Copy link

vercel bot commented Nov 2, 2023

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

Name Status Preview Comments Updated (UTC)
cosmos ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 3, 2023 3:09pm
swap-dev ✅ Ready (Inspect) Visit Preview Nov 3, 2023 3:09pm
widget-configurator ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 3, 2023 3:09pm

@alfetopito alfetopito marked this pull request as ready for review November 2, 2023 16:45
@alfetopito alfetopito requested a review from a team November 2, 2023 16:46
@alfetopito alfetopito force-pushed the permit/load-pre-generated-permit-info branch from e9df6b2 to 15ad18f Compare November 3, 2023 14:52
Base automatically changed from permit/load-pre-generated-permit-info to develop November 3, 2023 15:03
@alfetopito alfetopito force-pushed the permit/remove-feature-flags branch from 9119e6b to c7d15f5 Compare November 3, 2023 15:04
@alfetopito alfetopito merged commit 372b1a6 into develop Nov 3, 2023
4 of 5 checks passed
@alfetopito alfetopito deleted the permit/remove-feature-flags branch November 3, 2023 15:09
@github-actions github-actions bot locked and limited conversation to collaborators Nov 3, 2023
Copy link
Contributor

@anxolin anxolin left a comment

Choose a reason for hiding this comment

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

Great

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants