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

Feat/study closing: Add option to close a study #601

Merged
merged 42 commits into from
Jun 19, 2024
Merged

Conversation

hig-dev
Copy link
Contributor

@hig-dev hig-dev commented Apr 3, 2024

This PR adds the feature to close a study. It is not possible to re-open a study. No new participants will be able to enroll in a closed study.

  • Database: Add "is_closed" column to study table
  • Database: Add policy to restrict joining a closed study
  • App: Do not show closed studies in study selection
  • App: Show dialog if user tries to join a closed study
  • Designer: Use study status to show state of study (running, draft, closed)
  • Designer: Adapt study duplicate function
  • Designer: Add feature to close study using modal actions
  • Designer: Add banner to recruit page for closed study

@github-actions github-actions bot added documentation Improvements or additions to documentation app core designer_v2 labels Apr 3, 2024
Copy link

github-actions bot commented Apr 3, 2024

Visit the preview URL for this PR (updated for commit a4c34cc):

(expires Wed, 26 Jun 2024 11:11:29 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2149dad49ed83535217e50d5c18c0c8c90da629b

Copy link

github-actions bot commented Apr 3, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ REPOSITORY git_diff yes no 0.06s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

database/studyu-schema.sql Outdated Show resolved Hide resolved
database/studyu-schema.sql Outdated Show resolved Hide resolved
database/migration/migrate-close_study.sql Outdated Show resolved Hide resolved
core/lib/src/models/tables/study.dart Outdated Show resolved Hide resolved
core/lib/src/models/tables/study.dart Outdated Show resolved Hide resolved
core/lib/src/models/tables/study.dart Outdated Show resolved Hide resolved
database/migration/migrate-close_study.sql Outdated Show resolved Hide resolved
database/studyu-schema.sql Outdated Show resolved Hide resolved
database/studyu-schema.sql Outdated Show resolved Hide resolved
designer_v2/lib/domain/study.dart Outdated Show resolved Hide resolved
designer_v2/lib/localization/app_en.arb Outdated Show resolved Hide resolved
designer_v2/lib/localization/app_en.arb Outdated Show resolved Hide resolved
designer_v2/lib/localization/app_en.arb Outdated Show resolved Hide resolved
@johannesvedder johannesvedder force-pushed the feat/study-closing branch 2 times, most recently from 94339cd to 9fd7728 Compare May 27, 2024 20:34
johannesvedder and others added 26 commits June 11, 2024 15:28
# 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
#	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/recruit/study_recruit_page.dart
#	designer_v2/lib/features/study/study_status_badge.dart
#	designer_v2/lib/repositories/study_repository.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
feat: Introduce study status and add closing state
@johannesvedder johannesvedder merged commit e5ae24a into dev Jun 19, 2024
8 checks passed
@johannesvedder johannesvedder deleted the feat/study-closing branch June 19, 2024 11:11
@johannesvedder johannesvedder mentioned this pull request Jul 1, 2024
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants