-
Notifications
You must be signed in to change notification settings - Fork 0
feat: feature flags + projects message - in-page #519
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
Conversation
* feat: feature-flag–based error message (!) (!) style and layout incomplete, just testing * fix: change error message, link to news * Adds styles and addresses right margin issue * Linting --------- Co-authored-by: Sophia Massie-Perez <smassie@smassie-mbp25.local>
sophia-massie
left a comment
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.
LGTM!
| export default defineConfig({ | ||
| root: __dirname, | ||
| cacheDir: '../../node_modules/.vite/tup-ui', | ||
| // envPrefix: 'VITE_', |
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.
Do you want to leave this comment in here?
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.
Yes/No. I want us noobs to know that VITE_ (default value) has a special meaning. Without that prefix, the value is not available to client app. And I'd like to suggest we can change it. I could enable it and comment the doc URL —
| // envPrefix: 'VITE_', | |
| envPrefix: 'VITE_', // https://vite.dev/config/shared-options.html#envprefix |
— thoughts?
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.
OK perfect I like this
taoteg
left a comment
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.
LGTM.
|
Closed in favor of #522. |
Overview
Add feature flag and warn about maintenance.
Important
Credit to Sophia for helping style message.
Related
Changes
.envfor flagsTesting
UI
page.messages.mov