Skip to content
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

Redirect URL /records to /medical-records (when MR phase 1 reaches 100%) #88855

Open
1 task
sterkenburgsara opened this issue Jul 19, 2024 · 12 comments
Open
1 task
Assignees
Labels
blocked Cartographers MHV on VAgov team engineering Engineering topics ia

Comments

@sterkenburgsara
Copy link
Contributor

sterkenburgsara commented Jul 19, 2024

Description

When MR reaches 100%, the temp MR landing page will have no purpose and we need to redirect this page to the correct one.

User story

As a Veteran, I want to be able to get to Medical Records on VA.gov if I click an old URL that I have saved in a bookmark or elsewhere, so that I can still continue on and manage my care.

Notes

  • UPDATE: as of 11/25, teams are expected to do our own redirects. see slack post
  • Estimated timing: MR Ph1 staging review is Dec 18, should launch soon after actual launch was 12/20.

Possible tasks:

  • Read VA Platform: VA.gov to VA.gov Redirect
  • Confirm with Sitewide Content, Accessibility, Information Architecture team (CAIA) that our team puts in a request to another team rather than open our own PR

Acceptance criteria

  • URL va.gov/my-health/records/ redirects to va.gov/my-health/medical-records
  • QA review
@sterkenburgsara sterkenburgsara added Cartographers MHV on VAgov team my-health former "health-apartment" tag - Migrating MHV to va.gov needs-refinement Identifies tickets that need to be refined ia labels Jul 19, 2024
@wesrowe
Copy link
Contributor

wesrowe commented Aug 21, 2024

@sterkenburgsara, are you sure that CAIA needs to be involved in this redirect? Carlos thinks we could do it ourselves easily.

@carlosfelix2 carlosfelix2 changed the title Redirect URL /records to /medical-records (when MR reaches 100%) Tech Debt: Redirect URL /records to /medical-records (when MR reaches 100%) Aug 28, 2024
@carlosfelix2 carlosfelix2 changed the title Tech Debt: Redirect URL /records to /medical-records (when MR reaches 100%) Tech Debt: Redirect URL /records to /medical-records (when MR phase 1 reaches 100%) Aug 28, 2024
@wesrowe wesrowe added star-cartographer and removed my-health former "health-apartment" tag - Migrating MHV to va.gov star-cartographer labels Nov 19, 2024
@wesrowe wesrowe moved this to Next Refinement in Cartographers / MHV Home Nov 21, 2024
@wesrowe wesrowe moved this from Backlog to Next Refinement in Cartographers / MHV Home Dec 11, 2024
@wesrowe wesrowe added the engineering Engineering topics label Dec 11, 2024
@wesrowe wesrowe changed the title Tech Debt: Redirect URL /records to /medical-records (when MR phase 1 reaches 100%) Redirect URL /records to /medical-records (when MR phase 1 reaches 100%) Dec 11, 2024
@kjsuarez kjsuarez moved this from Next Refinement to Ready in Cartographers / MHV Home Dec 11, 2024
@mcarterA6 mcarterA6 removed the needs-refinement Identifies tickets that need to be refined label Dec 11, 2024
@wesrowe wesrowe moved this from Ready to Next Sprint in Cartographers / MHV Home Dec 17, 2024
@kjsuarez kjsuarez moved this from Next Sprint to Current Sprint in Cartographers / MHV Home Dec 26, 2024
@kjsuarez
Copy link
Contributor

CAIA redirect request

@kjsuarez kjsuarez self-assigned this Dec 26, 2024
@kjsuarez kjsuarez moved this from Current Sprint to In Progress in Cartographers / MHV Home Dec 26, 2024
@wesrowe
Copy link
Contributor

wesrowe commented Dec 26, 2024

@kjsuarez, I see your CAIA request ticket. Are you moving into implementation? The funny thing here is that Sara is our IA specialist, so you don't need CAIA's approval to implement the redirect. [edit:] Mikki clarified that CAIA is a PR reviewer for all redirects. (Still not quite sure if we needed to trigger any additional review with the ticket above; but I don't expect us to be doing these very often so no big deal.)

EDIT: I see that the notes in the ticket body are clear, but the possible task is not as accurate. Or rather, we already confirmed we have to actually do the implementation.

@kjsuarez
Copy link
Contributor

@wesrowe I understand that I need to do the implementation. Based on the documentation referenced in the slack post linked above, it seemed like we needed a CAIA approval before implementation. If that's not the case I'll go ahead and make the code change now.

@kjsuarez
Copy link
Contributor

PR

@kjsuarez kjsuarez moved this from In Progress to Code/PR Review in Cartographers / MHV Home Dec 26, 2024
@mnorthuis
Copy link
Contributor

@wesrowe FYSA, all redirects need approval from Sitewide IA (currently CAIA) - that is the team that vets and approves, regardless of the reason its being done. We are also approvers on the PR. This is to ensure the redirect is necessary, is documented, is valid, and kicks off any content work needed...among other reasons.

@kjsuarez @sterkenburgsara I approved the PR so you can move forward. We don't review code, so just wanted to drop a note to make sure all child URLs are being handled appropriately so there won't be any other pages with the /records sub-directory.

@kjsuarez kjsuarez moved this from Code/PR Review to QA/A11y Review in Cartographers / MHV Home Dec 30, 2024
@mcarterA6
Copy link

@allanto-ah - Would you be able to validate this in the appropriate environment?

@allanto-ah
Copy link
Contributor

@mcarterA6 Yes I should be able to do this in staging

@sterkenburgsara
Copy link
Contributor Author

Confirmed there are no child URLs under this directory. Only the /records page needs to be redirected.

@mcarterA6
Copy link

Need to confirm when this will be deployed.
@wesrowe will check with the public websites team to confirm deployment schedule.

@mcarterA6
Copy link

Marked as blocked until we confirm when this will be moved to production.

@wesrowe
Copy link
Contributor

wesrowe commented Jan 2, 2025

Friend of the team Jill Adams pointed me to this deployment log for the devops repo. It appears to have been releasing over the holiday break, but we would probably have to look deeper to see if Kevin's PR went.

I just tried in staging, the redirect is NOT in place there yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Cartographers MHV on VAgov team engineering Engineering topics ia
Projects
Status: QA/A11y Review
Development

No branches or pull requests

6 participants