Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

Terra Clinical Data Grid - Added Deprecation Notice #931

Merged
merged 2 commits into from
Jan 25, 2024
Merged

Conversation

mjpalazzo
Copy link
Contributor

Summary

What was changed:
Added deprecation notice and badge to terra clinical data grid

Why it was changed:

Terra clinical data grid does not comply with accessibility guidelines and was replaced by flowsheet data grid

Testing

This change was tested using:

  • WDIO
  • Jest
  • Visual testing (please attach a screenshot or recording)
  • Other (please describe below)
  • No tests are needed

Reviews

In addition to engineering reviews, this PR needs:

  • UX review
  • Accessibility review
  • Functional review
  • Tech writer review

Additional Details

This PR resolves:

UXPLATFORM-9930


Thank you for contributing to Terra.
@cerner/terra

Copy link

@RachaelMarso RachaelMarso left a comment

Choose a reason for hiding this comment

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

Reviewed and approved per OHAIUAD-206539]

@sycombs
Copy link
Contributor

sycombs commented Jan 25, 2024

@mjpalazzo Can you rename the PR to something like "[terra-clinical-data-grid] Added deprecation notice" so that the commit message will reflect the change?

@mjpalazzo mjpalazzo changed the title Mjpalazzo patch1 Terra Clinical Data Grid - Added Deprecation Notice Jan 25, 2024
@mjpalazzo
Copy link
Contributor Author

@mjpalazzo Can you rename the PR to something like "[terra-clinical-data-grid] Added deprecation notice" so that the commit message will reflect the change?

Thanks @sycombs - renamed as you recommended

@mjpalazzo mjpalazzo requested a review from sycombs January 25, 2024 19:05
@mjpalazzo mjpalazzo removed the request for review from adoroshk January 25, 2024 19:09
@mjpalazzo mjpalazzo marked this pull request as ready for review January 25, 2024 19:09
@mjpalazzo mjpalazzo merged commit 33a6bad into main Jan 25, 2024
22 checks passed
@mjpalazzo mjpalazzo deleted the mjpalazzo-patch1 branch January 25, 2024 19:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants