-
Notifications
You must be signed in to change notification settings - Fork 429
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
fix: The organisation setting page is broken locally #4330
fix: The organisation setting page is broken locally #4330
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Docker builds report
|
Uffizzi Preview |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These don't look right to me based on our conversation @kyle-ssg ?
@matthewelwell , I'm sorry, maybe I'm missing something, but this is not related to the announcement flag. It's related to changes in the Payment file, for example: |
@@ -16,6 +16,10 @@ module.exports = global.Project = { | |||
flagsmithClientEdgeAPI: 'https://edge.api.flagsmith.com/api/v1/', | |||
// This is used for Sentry tracking | |||
maintenance: false, | |||
plans: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
plans: {
scaleUp: { annual: 'scale-up-annual-v2', monthly: 'scale-up-v2' },
startup: { annual: 'startup-annual-v2', monthly: 'startup-v2' },
},
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@novakzaballa this is what I was getting at.
Thanks for submitting a PR! Please check the boxes below:
pre-commit
to check lintingdocs/
if required so people know about the feature!Changes
plans
key in the project_local fileHow did you test this code?