-
Notifications
You must be signed in to change notification settings - Fork 6
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
Feature: Flow states, about v2 #7800
base: develop
Are you sure you want to change the base?
Conversation
…alkem-io/client-web into client-7685-innovationFlow-tabs
…flowState to space tabs
…Collaboration hook; other misc refactoring; there are bugs introduced with this commit - but from here can start to stabilise
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Only use flowState for the filtering of Callout.
Removed all groups related functionality.
New hook to get the information needed for each of the tabs, including the state information that is then usable for both the tab name and the description.
Moved code around to have the tabbed pages as one layout under the new Space location.
Removed a couple of containers (merged into pages), and also a couple of hooks.
Updated to use refined api on the server.
Updated to use explicit passing around of classificationTagsets as an array
.....
Risk areas / TODOs:
======
#7685
#6937
Disclaimer: There are certain hacks here to make the callouts be shown in the correct group when they come with certain flowState set that should NOT be merged. But serve as a proof of concept and display quite well the issue.
Problem:
tagsets
that are incallout > framing > profile > tagsets
.tagset.name = 'callout-group'
. and there is a full server side implementation to filter and move callouts in between groups.tagset.name = 'flow-state'
. and we have a nice client-side implementation to move callouts between states with a drag&drop editor.Second Problem:
CalloutGroupName
, we need to give flexibility to that, we can keep the 4/5 possible urls but I think we should unify DashboardPage, CommunityPage, SubspacesPage and KnowledgeBasePage and I think they should be a single one that renders the specific components for each section.Server Actions:
callouts (groupNames[...]) {
to be able to query callouts by state.Pending - things that may not be ver difficult and can be continued regardless of server changes
currentState
tab when we don't specify a tab in the url