-
Notifications
You must be signed in to change notification settings - Fork 0
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/sckan 333 #360
Feature/sckan 333 #360
Conversation
backend/composer/api/filtersets.py
Outdated
Q(user=system_user, note__icontains="exported") | | ||
~Q(user=system_user) | ||
~Q(user=system_user)) & | ||
~Q(type__in=[NoteType.TRANSITION]) |
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.
This allows us to - not show any transition
related notes by default
@D-GopalKrishna an issue here - the export transition should show up as per acceptance criteria scrnli_XAuCM5d7WaQ1mi.webm |
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.
exported missing the filter
- to include invalid and exported transitions
thanks @ddelpiano ! Updated it - by removing the logic to - not include the transition notes. 2024-11-26.16-02-20.mp4 |
Jira link - SCKAN-333
About task:
demo video
Screencast from 11-17-2024 03:26:57 PM.webm