Skip to content

v2 questionnaires; update dependencies#155

Draft
lukaskollmer wants to merge 3 commits into
mainfrom
lukas/v2-questionnaires
Draft

v2 questionnaires; update dependencies#155
lukaskollmer wants to merge 3 commits into
mainfrom
lukas/v2-questionnaires

Conversation

@lukaskollmer

Copy link
Copy Markdown
Collaborator

v2 questionnaires; update dependencies

⚙️ Release Notes

  • updated a bunch of dependencies
  • added a debug toggle to try the new questionnaire UI & infrastructure

📚 Documentation

todo

✅ Testing

todo

Code of Conduct & Contributing Guidelines

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

@coderabbitai

coderabbitai Bot commented Apr 14, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e76a4433-ff17-472d-aa03-e8d3154e014f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch lukas/v2-questionnaires

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 and usage tips.

@PSchmiedmayer PSchmiedmayer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Did a first pass; let me know once the PR is ready to be merged for a full review; thanks for working on all of this 🚀

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

TODO to use tagged versions here before the merge.


var body: some View {
Group {
Group { // TODO use a `VStack(spacing: 0)` here?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Resolve TODO.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice, thank you for adding the abstraction here!

import class ModelsR4.Questionnaire
import MyHeartCountsShared
import ResearchKitSwiftUI
import ResearchKitSwiftUI // TODO WHY????

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

TODO to be resolved.

import MHCStudyDefinition
import OSLog
import ResearchKitSwiftUI
import ResearchKitSwiftUI // TODO WHY????

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

TODO to be resolved.

Comment on lines +116 to +117
fatalError("TODO")
// try await self.init(SpeziFHIR.FHIRResource.initialize(basedOn: record, using: healthKit, loadHealthKitAttachments: true))

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

TODO to be resolved.

} catch {
await logger.error("Failed to convert QuestionnaireResponses to FHIR: \(error)")
}
// // shouldn't be necessary, but we had some issues with these not being properly set

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Do we still need that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants