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

Add Notes to the timeline view #2715

Closed
rahul1 opened this issue Aug 24, 2023 · 1 comment
Closed

Add Notes to the timeline view #2715

rahul1 opened this issue Aug 24, 2023 · 1 comment
Assignees
Labels
good first issue Good for newcomers react Features, fixes and enhancements to react component library

Comments

@rahul1
Copy link
Member

rahul1 commented Aug 24, 2023

Currently, we have a few react components for displaying timeline views:

Most of these components do some kind of search for resources related to the root resource, and then display them in chronological order. For instance, many search for Communication resources that are linked to the root resource, which are treated as "comments".

However, many resources also contain a note field on them, that could also be interpreted as "comments". Some examples:

  • Observation
  • ServiceRequest
  • MediationRequest
  • ...

The request for this issue is to modify the appropriate timeline components to:

  1. Check if the root resource has a note element
  2. If so, add it to the timeline
@rahul1 rahul1 added react Features, fixes and enhancements to react component library good first issue Good for newcomers labels Aug 24, 2023
@ThatOneBro ThatOneBro added this to the August 31, 2023 milestone Aug 25, 2023
@codyebberson
Copy link
Member

Discussed offline, the complexity to support this did not seem worth the benefit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers react Features, fixes and enhancements to react component library
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants