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

Multiple Instructions should be supported as per specification #2092

Closed
khyativyasargus opened this issue Jul 21, 2023 · 2 comments · Fixed by #2098
Closed

Multiple Instructions should be supported as per specification #2092

khyativyasargus opened this issue Jul 21, 2023 · 2 comments · Fixed by #2098
Assignees
Labels
P1 High priority issue ux

Comments

@khyativyasargus
Copy link
Contributor

khyativyasargus commented Jul 21, 2023

Describe the bug
Currently, with Questionnaire Header, only the first instruction is displayed. But as per FHIR specifications, it should support multiple instructions.

Component
SDC library

To Reproduce
Steps to reproduce the behavior:

  1. Go to HeaderView.kt
  2. Observe the integration of the hint view. It only supports single instruction.
  3. Same for GroupHeaderView.kt and other places

Expected behavior
As per FHIR specifications, Instructions can be 0..*, so multiple hints should be supported.
Reference for issue - #Smart-emcare issue

Would you like to work on the issue?
Please state if this issue should be assigned to you or who you think could help to solve this issue.

@santosh-pingle

@santosh-pingle santosh-pingle self-assigned this Jul 24, 2023
@santosh-pingle santosh-pingle added P1 High priority issue ux labels Jul 24, 2023
@santosh-pingle
Copy link
Collaborator

I will take a look, thanks!

@santosh-pingle
Copy link
Collaborator

@khyativyasargus
Can you please attach the questionnaire.json for the reference?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 High priority issue ux
Projects
Status: Complete
Development

Successfully merging a pull request may close this issue.

2 participants