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

Add side panel in lab #10183

Merged
merged 1 commit into from
Feb 14, 2025
Merged

Add side panel in lab #10183

merged 1 commit into from
Feb 14, 2025

Conversation

thomtrp
Copy link
Contributor

@thomtrp thomtrp commented Feb 13, 2025

Capture d’écran 2025-02-13 à 11 15 22

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Added a new feature flag to enable the side panel functionality through the Labs section, allowing controlled rollout of this experimental feature.

  • Added IsCommandMenuV2Enabled to PublicFeatureFlag type in /packages/twenty-server/src/engine/core-modules/feature-flag/constants/public-feature-flag.const.ts
  • Feature will be accessible through the Labs tab in settings, following the pattern established in release 0.41.0

1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

@thomtrp thomtrp enabled auto-merge (squash) February 13, 2025 10:26
@thomtrp thomtrp disabled auto-merge February 13, 2025 10:27
Copy link
Contributor

@bosiraphael bosiraphael left a comment

Choose a reason for hiding this comment

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

Waiting on my PR before merging this

Copy link
Contributor

@bosiraphael bosiraphael left a comment

Choose a reason for hiding this comment

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

LGTM

@thomtrp thomtrp force-pushed the tt-setup-side-panel-in-labs branch from 1a5998b to 4371fbf Compare February 14, 2025 13:53
@thomtrp thomtrp enabled auto-merge (squash) February 14, 2025 13:53
@thomtrp thomtrp merged commit 5fe1015 into main Feb 14, 2025
32 checks passed
@thomtrp thomtrp deleted the tt-setup-side-panel-in-labs branch February 14, 2025 13:59
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.

3 participants