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 opened section #7265

Merged
merged 6 commits into from
Oct 7, 2024
Merged

Conversation

thomtrp
Copy link
Contributor

@thomtrp thomtrp commented Sep 26, 2024

When object is not part of the workspace favorite list, we want to show it in the "opened section" while its record page is accessed.

This PR:

  • adds a new component NavigationDrawerOpenedSection
  • makes workflow versions and runs not system object + creates a prefilled view index for these
  • do not create workspace favorites for these so these do not appear in the workspace section
Capture d’écran 2024-09-26 à 11 45 25

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

This pull request introduces an "opened section" in the navigation drawer for objects that are not part of the workspace favorite list. Here's a summary of the key changes:

  • Added new NavigationDrawerOpenedSection component to display objects not in workspace favorites
  • Created useObjectMetadataItemsInWorkspaceFavorites hook to efficiently manage workspace favorites
  • Modified data seeding process to exclude certain objects (workflow runs and versions) from workspace favorites
  • Introduced new views for workflow runs and versions, making them non-system objects
  • Updated WorkflowRunWorkspaceEntity and WorkflowVersionWorkspaceEntity to support the new navigation structure

Key points:

  • New getExcludedObjectMetadataIdsForFavoritesPrefill function controls which objects appear in workspace favorites
  • WorkspaceFavorites component simplified to use the new custom hook
  • Standard field IDs updated to include new fields for workflow runs and versions
  • Conditional rendering of the new opened section based on feature flag

15 file(s) reviewed, 4 comment(s)
Edit PR Review Bot Settings

@thomtrp thomtrp force-pushed the tt-add-navigation-item-for-active-view-if-not-exist branch from 81ec221 to 4da2d4f Compare October 1, 2024 13:38
Copy link
Member

@charlesBochet charlesBochet 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-add-navigation-item-for-active-view-if-not-exist branch from 539bade to 386b7bd Compare October 7, 2024 09:46
@thomtrp thomtrp force-pushed the tt-add-navigation-item-for-active-view-if-not-exist branch from 386b7bd to a92f84e Compare October 7, 2024 09:46
@thomtrp thomtrp force-pushed the tt-add-navigation-item-for-active-view-if-not-exist branch from a92f84e to d165114 Compare October 7, 2024 09:57
@thomtrp thomtrp merged commit ce676f6 into main Oct 7, 2024
15 of 16 checks passed
@thomtrp thomtrp deleted the tt-add-navigation-item-for-active-view-if-not-exist branch October 7, 2024 11:45
harshit078 pushed a commit to harshit078/twenty that referenced this pull request Oct 14, 2024
When object is not part of the workspace favorite list, we want to show
it in the "opened section" while its record page is accessed.

This PR:
- adds a new component `NavigationDrawerOpenedSection`
- makes workflow versions and runs not system object + creates a
prefilled view index for these
- do not create workspace favorites for these so these do not appear in
the workspace section

<img width="1129" alt="Capture d’écran 2024-09-26 à 11 45 25"
src="https://github.com/user-attachments/assets/c84d773c-0bef-4dce-b66a-55d7d00b0fb6">
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.

2 participants