-
Notifications
You must be signed in to change notification settings - Fork 9
feat(cat-voices): workspace filtering and tabs #3765
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
📚 Docs PreviewThe docs for this PR can be previewed at the following URL: https://docs.dev.projectcatalyst.io/voices/feat/workspace-tabs-3692 |
damian-molinski
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.
Love how you setup widgets but had few comments around bloc
catalyst_voices/apps/voices/lib/pages/workspace/page/workspace_page.dart
Outdated
Show resolved
Hide resolved
catalyst_voices/apps/voices/lib/pages/workspace/widgets/workspace_proposal_filters.dart
Outdated
Show resolved
Hide resolved
catalyst_voices/apps/voices/lib/pages/workspace/widgets/workspace_proposal_filters.dart
Outdated
Show resolved
Hide resolved
catalyst_voices/apps/voices/lib/pages/workspace/widgets/workspace_tabs.dart
Outdated
Show resolved
Hide resolved
catalyst_voices/packages/internal/catalyst_voices_blocs/lib/src/workspace/workspace_bloc.dart
Outdated
Show resolved
Hide resolved
catalyst_voices/packages/internal/catalyst_voices_blocs/lib/src/workspace/workspace_bloc.dart
Outdated
Show resolved
Hide resolved
catalyst_voices/packages/internal/catalyst_voices_blocs/lib/src/workspace/workspace_bloc.dart
Outdated
Show resolved
Hide resolved
catalyst_voices/packages/internal/catalyst_voices_blocs/lib/src/workspace/workspace_bloc.dart
Outdated
Show resolved
Hide resolved
...t_voices/packages/internal/catalyst_voices_blocs/lib/src/workspace/workspace_bloc_cache.dart
Outdated
Show resolved
Hide resolved
...es/packages/internal/catalyst_voices_view_models/lib/src/proposal/user_proposal_invites.dart
Outdated
Show resolved
Hide resolved
.../voices/lib/pages/workspace/widgets/user_proposal_invites/user_proposal_invites_section.dart
Show resolved
Hide resolved
catalyst_voices/packages/internal/catalyst_voices_blocs/lib/src/workspace/workspace_state.dart
Outdated
Show resolved
Hide resolved
damian-molinski
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
* feat: working integration * chore: use bloc instead of cubit * chore: final changes * chore: selfreview * feat: review update * fix: readme * chore: rename factory constructor * feat: merge collaborators status enum into one
Description
This PR adds UI to ad tabs in workspace page for proposals and invitations also adds filters to filter proposal base on if user is an author or a collaborator.
Bloc is not fully intergrated with a service at this stage as it was not clear how exactly will this data be formatted at. Will do in another PR as this is kinda big. But logic for chaning filters and tabs is implemented.
Related Issue(s)
Closes #3692
Screenshots
Screen.Recording.2025-11-25.at.11.57.00.mov
Please confirm the following checks