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
281 stars 197 forks source link

[Design] Finalize appt detail pages #89874

Closed benbrasso-agile6 closed 2 weeks ago

benbrasso-agile6 commented 1 month ago

Tasks

Notes on what to remove and add: https://docs.google.com/document/d/1_QB_4hBYQYATyw7Fj2GxNB2sxptf5S_i8p-EG3UexwY/edit?usp=sharing

benbrasso-agile6 commented 1 month ago

Please add your planning poker estimate with Zenhub @zach-park

loripusey commented 1 month ago

@zach-park please point

zach-park commented 1 month ago

@cgednalske , fyi.

zach-park commented 1 month ago

Figma file link (in progress): https://www.figma.com/design/7Ib7RxiIC4QB53FDBO2a8c/Unified-check-in-%7C-PCI?node-id=2645-15044&t=99yKDeWLvcOeh4An-4

loripusey commented 2 weeks ago

@benbrasso-agile6 @zach-park when we finish these wireframes, please post in our team channel and ping frontend, why will need them to point their work ticket

zach-park commented 2 weeks ago

@benbrasso-agile6 @cgednalske, ready for validation: PCI: https://www.figma.com/design/mVg6S9xgiQpWbAOAvptQOZ/Pre-check-in-%7C-PCI?node-id=543-9595&t=g9M44JXLce1xwwEZ-1 CI (duplicate of PCI): https://www.figma.com/design/XYoHvDxgB2BpZIo2rrhFba/Check-in-%7C-PCI?node-id=4254-10430&t=vKSs5ZQPHGjFXTy2-1

benbrasso-agile6 commented 2 weeks ago

Looks good. @loripusey I'm going to close this for now. Whenever Kanchana finishes her work, we can create a new ticket to update these. I don't think FE would want to start these, nor do I think you're going to want it to be a priority. Or, maybe you do. Anyways, check them out:

Pre-check-in appt detail pages (for upcoming appts) check-in appt detail pages (for upcoming appts)

@zach-park nicely done

benbrasso-agile6 commented 2 weeks ago

@outerpress do you mind look at these? I think you've been moving the headings around on the detail pages. Do we have these correct? Or do we need to wait a bit for your work to catch up?

Pre-check-in appt detail pages (for upcoming appts) check-in appt detail pages (for upcoming appts)

Note that most, if not all, actions are removed because of limitations with low auth vs. full auth.

outerpress commented 1 week ago

@benbrasso-agile6 looks good! I dropped a couple of comments. Updates to the "other actions" section are still WIP so nothing to add from that here.

benbrasso-agile6 commented 1 week ago

appreciate it, tysm!

benbrasso-agile6 commented 1 week ago

Updates are all set 👍

Pre-check-in appt detail pages (for upcoming appts) check-in appt detail pages (for upcoming appts)