-
Notifications
You must be signed in to change notification settings - Fork 214
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
[UX] Update CC referral error alert content to refer to specific referral #100808
Comments
@loripusey @StevenStraily @zach-park I updated the information and tasks here a bit to better reflect what we discussed and include adding the requests and referrals link to the designs that are missing it. |
@zach-park check out the tasks in this ticket. I edited the language, which should clarify things. Can't say I have any more information to add. |
Hi @christina, I have a quick question. Is this the Referrals and requests link (attached below)? |
Yep, that's it 👍 It's just missing from our error pages, but should be like the MVP pages above. |
Note that that is a separate, random task from updating the error - just because we realized it wasn't there and it was causing confusion. |
Hi @cgednalske @StevenStraily (cc: @kristenmcconnell ), this ticket is ready for review: Also, should I add this item for review during the SME meeting tomorrow? Thank you! |
Hi @zach-park I think you might have this work in the wrong ticket. Sounds like the one you just completed was this one related to the page heading? [UX] Evaluate CC referral error page heading #100796 Or maybe that one and this one kind of fed into each other and your changes will end up working towards both. Either way, yes, bringing this to the SME sync sounds good. Also because this might affect other Appointments error pages 🙂 |
Yes, I believe these two tickets (mentioned above) feed into each other. :) |
Looks like the requirements for the ticket are met to me. Do you feel like we need any follow-ups here @zach-park? |
@zach-park I'm not sure the correct alert has been addressed here. It should be this one below that needs updating, and it doesn't look like it refers to the specific referral at the moment. Currently, the alert reads "referral info" in general. How can this be slightly more specific? |
Sorry, but I'm away until the afternoon. But I will follow up later in the evening. Apologies!! |
Hi @cgednalske , my understanding from John was that this page (e.g., Can't get referral ID) wouldn't be affected by the general error. That was the reason it wasn't addressed. But it might have been an oversight so I will confirm with @wullaski on Monday. |
Background
When the UUID for the specific referral cannot be fetched, an error alert appears instead of the referral card. Currently our language in the alert is generic and could be mistaken for any/all referrals instead of that specific referral. According to Brian, referrals list on the Referrals and requests page would still be available.
Figma
Note: additionally, our error screens in the MVP scope page are not currently accurate. The designs need to be updated to include the Referrals and requests link like all other pages.
Tasks
Tasks
The text was updated successfully, but these errors were encountered: