department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
282 stars 202 forks source link

Deduplicate reason and patient comments rendering logic #94339

Open JunTaoLuo opened 1 week ago

JunTaoLuo commented 1 week ago

Summary

WIP, we'll need to take a look at this with Tony when he's back to get a better understanding of what's needed here.

Currently noticing duplicated logic in the following places which may lead to rendering inconsistent information:

Specs

Steps to Reproduce

Test User:

  1. Go to: URL

Actual Result

Describe what happened that was unexpected or undesired; including screenshots is encouraged

Desired Result

Describe what was supposed to happen; reference existing Github issue that contains a solution if possible

Definition of Done


How to configure this issue

ldelacosta commented 2 days ago

@JunTaoLuo @vbahinwillit - if you can take a look at this ticket and let us know if there are additional work needed. cc: @simiadebowale @JRRoof (see thread)

Bren22va commented 1 day ago

Hey team! Please add your planning poker estimate with Zenhub @jenniemc @ryanshaw @simiadebowale @vbahinwillit

Bren22va commented 1 day ago

Please add your planning poker estimate with Zenhub @JunTaoLuo