Closed mnorthuis closed 3 years ago
Thanks @mnorthuis!
Choosing "requested" from drop list results in an error.
Is your recommendation to change the error language? Or more broadly about the error itself?
The "show by type" drop list appears to be a list of statuses, but "cancelled" (although it is a separate option) appointments show up in "upcoming"
We went this route because we were worried about cancelled appointments disappearing from the Veterans view, especially when they were not cancelled by the Veteran. We don't have a way to display a temporary update to a veteran when an appointment status changes yet, but we have talked about it.
and "past".
I can't test this now but there shouldn't be cancelled past appointments, since an appointment that was cancelled never happened. I think we ran into this issue a little while ago but fixed it. @laurenernest do you remember this one?
Results could be confusing to users when a "cancelled" appointment appears as the first option in the "upcoming" list.
Agreed. We'll keep an eye out for any confusion around this as we track the release.
If showing cancelled appointments in the other filters is important (understandably could be), an option could be showing "cancelled" as a separate control - like a toggle to include/exclude cancelled appointments in any of the other views. (108)
Just for added context, here's what we heard around cancelled appointments.
Consider using "Show by status" rather than "Show by type" for a more descriptive label. (109)
I don't have a way of knowing if one would be any clearer than the other to a veteran - do we use "status" elsewhere? @peggygannon any thoughts on this one?
Because each set of results is on a new page, it creates a situation where a new page is added to browsing history, but the new page has the same page title, and no other indicators on the page (i.e. breadcrumb, H1, etc) show that they have navigated to a new page. This may create some frustrations if the user is attempting to go back and needs to navigate through multiple variations of a page.
This makes sense. We could update the page title to something like [Your appointments/Past appointments/Requests/Canceled appointments] VA online scheduling | Veterans affairs
cc @laurenernest
It also creates an issue where once on a detail page, if the user uses the breadcrumb to go back, they go back to the default state of the page rather than to a potential filtered page they have been on prior. These are somewhat minor issues, but other implementation methods may create a cleaner experience for all users. (6,78)
Agreed that this is an issue. The detail page is one of those areas that (we may have talked about this) it might be better to not have a breadcrumb and just have a back button at the top of the page that would return to the previous context. This is in the back of my head as something we could test as a variant post-release.
I'll answer to the "Must" feedback item...if this is a known or intended error message due to a service failure, you can ignore this feedback. If this is a QA issue where a result was intended but something is failing, then it needs to be corrected.
Choosing "requested" from drop list results in an error. Actually had this issue with different statuses at different times. (11)
We’re sorry. We’ve run into a problem
We’re having trouble getting your appointment requests. Please try again later.
In regards to the other feedback, nothing is required to be changed, but should be reviewed for potential improvements.
Your suggestion to updating the H1 and page title to indicate the change in page when a different option is selected may help with clearing up browser history and back behavior.
I do have it on my list of wishes to explore instances where removing the breadcrumb and replacing it with a "Back" link may be helpful. I will add this one to my examples.
Choosing "requested" from drop list results in an error.
This was due to downstream system errors (we had a few issues the past week). The Judy user doesn't currently have any open requests so this is what it looks like.
and "past".
I was able to repro. we'll fix this.
@laurenernest everything's captured from this one on my end
@laurenernest @outerpress I agree with Mikki's suggestion to use "Show by status" for the drop down label. We use "type" of appt (video, in person, etc) throughout flow, so "status" is more accurate description for this scenario/label.
Thanks @peggygannon and @mnorthuis that makes sense to me!
included changes we're taking action on in #26657. closing ticket.
VFS acceptance criteria
Thoughts/questions
Feedback
Must:
Should:
Consider: