Skip to content

Integrate study and scheduler into mhc app - #24

Merged
PSchmiedmayer merged 1 commit into
mainfrom
task/mhc-home
Aug 20, 2026
Merged

Integrate study and scheduler into mhc app#24
PSchmiedmayer merged 1 commit into
mainfrom
task/mhc-home

Conversation

@eldcn

@eldcn eldcn commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

♻️ Current situation & Problem

  • Wires the study bundle and scheduler into the app for the first time, and builds the Home tab on top of them.
  • MHCStudyBundleProvider loads the bundle from assets on configure and hands it to the study manager on every launch, so an enrollment carries forward when the bundle describes a newer study revision.
  • StudyEnroller enrolls once at the final onboarding step, reusing the account's dateOfEnrollment when present so schedules survive a reinstall.
  • Consent now resolves from the bundle for the device locale, falling back to the bundled en-US asset — hence ConsentConfigurationBuilder.document becoming suspending.
  • The Home tab renders today's tasks, missed tasks, nudges and always-available tasks from scheduler events through the :ui-scheduler components.
  • Only the article action is handled so far: an informational task opens its markdown in a bottom sheet and completes on presentation. Questionnaires, timed walk/run tests and ECG raise their intent but do nothing yet; the missed-tasks list, prompted actions and nudge sources follow in separate PRs.
  • Integrate gradle task to fetch the released study bundle artifact and store it app assets
Screenshot_1785772927

⚙️ Release Notes

Add a bullet point list summary of the feature and possible migration guides if this is a breaking change so this section can be added to the release notes.
Include code snippets that provide examples of the feature implemented or links to the documentation if it adds to or changes the public interface.

📚 Documentation

Please ensure that you properly document any additions in conformance with the project's documentation guidelines.
You can use this section to describe your solution, but we encourage contributors to document your reasoning and changes using inline documentation.

✅ Testing

Please ensure that the PR meets the testing requirements set by Codecov and that new functionality is appropriately tested.
This section describes important information about the tests and why some elements might not be testable.

Code of Conduct & Contributing Guidelines

By creating and submitting this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

@eldcn
eldcn requested a review from PSchmiedmayer August 8, 2026 13:52
@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 13587dca-1a77-404b-8af8-97c0ac13933a


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@eldcn
eldcn force-pushed the task/mhc-home branch 2 times, most recently from 17dffbd to 4146b0b Compare August 8, 2026 14:35
@codecov

codecov Bot commented Aug 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 58.82353% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...i/scheduler/SchedulerNotificationsConfiguration.kt 0.00% 7 Missing ⚠️

📢 Thoughts on this report? Let us know!

@PSchmiedmayer PSchmiedmayer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the work here @eldcn; would let @pauljohanneskraft some insights but happy to merge for now as we go through changes step by step.

@PSchmiedmayer
PSchmiedmayer merged commit 9f49738 into main Aug 20, 2026
19 checks passed
@PSchmiedmayer
PSchmiedmayer deleted the task/mhc-home branch August 20, 2026 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants