-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: Introduce study status and add closing state #634
feat: Introduce study status and add closing state #634
Conversation
Visit the preview URL for this PR (updated for commit 931c4e2):
(expires Fri, 21 Jun 2024 07:38:19 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 2149dad49ed83535217e50d5c18c0c8c90da629b |
235acd5
to
e4fae75
Compare
94339cd
to
9fd7728
Compare
c643ed5
to
f312f3e
Compare
9fd7728
to
30b84ee
Compare
a0e2f15
to
3d4e0b6
Compare
bc5d39b
to
3d4e0b6
Compare
7048bbf
to
9b84100
Compare
Hi, I tried to launch a new study. after inputting all the values, when I press |
The loading widget on top is spinning which means the study hasn't been saved correctly. I have made some additions to the PR fixing different things. Please test again. |
9374962
to
c9c1fa6
Compare
c9c1fa6
to
7c5b8ae
Compare
# Conflicts: # core/lib/src/models/tables/study.dart # core/lib/src/models/tables/study_subject.dart # core/lib/src/util/supabase_object.dart # designer_v2/lib/common_views/action_popup_menu.dart # designer_v2/lib/features/publish/study_publish_dialog.dart # designer_v2/lib/features/recruit/study_recruit_page.dart # designer_v2/lib/features/study/study_scaffold.dart # designer_v2/lib/features/study/study_status_badge.dart # designer_v2/lib/services/notifications.dart # designer_v2/lib/utils/model_action.dart
# Conflicts: # core/lib/src/models/tables/study.dart # designer_v2/lib/common_views/action_popup_menu.dart # designer_v2/lib/features/recruit/study_recruit_page.dart # designer_v2/lib/repositories/study_repository.dart # designer_v2/lib/services/notifications.dart # designer_v2/lib/utils/model_action.dart
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.
All seems working ✅
- Checkout the branch and start a new database using the Supabase CLI for StudyU
- Log into the Designer and launch a new public study
- Log in with another user and check that they are unable to close the study from the study registry
- Open up the app and start the study as a participant
- Change to the first user again who owns the study and close the study
- Check in the app that the participant is still able to continue the study, even though the study has been closed
- Start a new session with the app and check that the study is not shown anymore in the study list, since it is closed
PR changes:
closed
as a state into thestudy
database model.published
has been deprecated.study
columns when study is not in draft mode, except'updated_at', 'status', 'registry_published',
and'result_sharing'
Possible Test cases to Review