Skip to content

Conversation

@jenny-heath
Copy link
Contributor

Link to pivotal/JIRA issue

Is PM acceptance required? (delete one)

  • Yes - don't merge until JIRA issue is accepted!

Reminder: merge main into this branch and get green tests before merging to main

What was done?

  • Paired with Anu through a bunch of copy and styling updates; didn't take screenshots of everything because we looked at it together
  • There's more to do but wanted to keep the PR short

How to test?

  • Mostly just looked at the pages to make sure they looked right
  • Fixed some failing tests that were expecting the old copy

Screenshots (for visual changes)

  • Lots of visual changes but they've already been looked at by design and should be reviewed one more time in acceptance testing

@github-actions
Copy link

github-actions bot commented Dec 3, 2024

Heroku app: https://gyr-review-app-5075-b2535846083a.herokuapp.com/
View logs: heroku logs --app gyr-review-app-5075 (optionally add --tail)

@jenny-heath jenny-heath merged commit fd3b382 into main Dec 3, 2024
4 checks passed
@jenny-heath jenny-heath deleted the copy-ui-updates branch December 3, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants