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

chore(dashboard,web): Clean up feature flags #7365

Merged
merged 2 commits into from
Dec 27, 2024

Conversation

SokratisVidros
Copy link
Contributor

What changed? Why was the change needed?

Cleaning up feature flags

Copy link

netlify bot commented Dec 24, 2024

Deploy Preview for dev-web-novu ready!

Name Link
🔨 Latest commit a3bb80d
🔍 Latest deploy log https://app.netlify.com/sites/dev-web-novu/deploys/676e88ad4a5816000884309f
😎 Deploy Preview https://deploy-preview-7365.dashboard.novu-staging.co
📱 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.

Copy link

netlify bot commented Dec 24, 2024

Deploy Preview for dashboard-v2-novu-staging ready!

Name Link
🔨 Latest commit a3bb80d
🔍 Latest deploy log https://app.netlify.com/sites/dashboard-v2-novu-staging/deploys/676e88ad415fe6000819a03d
😎 Deploy Preview https://deploy-preview-7365.dashboard-v2.novu-staging.co
📱 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.

Copy link

LaunchDarkly flag references

❌ 5 flags removed

Name Key Aliases found Info
IS_API_IDEMPOTENCY_ENABLED IS_API_IDEMPOTENCY_ENABLED ⚠️ not all references removed
IS_API_RATE_LIMITING_DRY_RUN_ENABLED IS_API_RATE_LIMITING_DRY_RUN_ENABLED ⚠️ not all references removed
IS_API_RATE_LIMITING_ENABLED IS_API_RATE_LIMITING_ENABLED ⚠️ not all references removed
IS_TEMPLATE_STORE_ENABLED IS_TEMPLATE_STORE_ENABLED ⚠️ not all references removed
IS_USE_MERGED_DIGEST_ID_ENABLED IS_USE_MERGED_DIGEST_ID_ENABLED ⚠️ not all references removed

@SokratisVidros SokratisVidros changed the title chore(dashboard,web): Clean up feature flags - DO NOT MERGE YET chore(dashboard,web): Clean up feature flags Dec 27, 2024

return (
<UserButton
afterSignOutUrl={`${WEB_APP_URL}${ROUTES.AUTH_LOGIN}`}
userProfileUrl={ROUTES.MANAGE_ACCOUNT_USER_PROFILE}
>
{isNewDashboardEnabled && (
Copy link
Contributor

Choose a reason for hiding this comment

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

This will also show this to v1 users right? I'm not sure how their experience will be 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They should see the button, go to the new Dashboard, see all their workflows but for V1 workflows go back to the old web app on workflow selection.

I will verify it on staging.

Copy link

pkg-pr-new bot commented Dec 27, 2024

Open in Stackblitz

@novu/client

npm i https://pkg.pr.new/novuhq/novu/@novu/client@7365

@novu/headless

npm i https://pkg.pr.new/novuhq/novu/@novu/headless@7365

@novu/node

npm i https://pkg.pr.new/novuhq/novu/@novu/node@7365

@novu/notification-center

npm i https://pkg.pr.new/novuhq/novu/@novu/notification-center@7365

novu

npm i https://pkg.pr.new/novuhq/novu@7365

@novu/providers

npm i https://pkg.pr.new/novuhq/novu/@novu/providers@7365

@novu/shared

npm i https://pkg.pr.new/novuhq/novu/@novu/shared@7365

commit: a3bb80d

@SokratisVidros SokratisVidros merged commit d692289 into next Dec 27, 2024
42 checks passed
@SokratisVidros SokratisVidros deleted the cleanup_feature_flags branch December 27, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants