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.

Sign up for GitHub

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

Jump to bottom

feat: Move "Change" button to section on Review component #2296

Closed
wants to merge 1 commit into from
Closed

feat: Move "Change" button to section on Review component #2296

wants to merge 1 commit into from

Conversation

DafyddLlyr
Copy link
Contributor

@DafyddLlyr DafyddLlyr commented Oct 11, 2023 •
edited
Loading

What does this PR do?

  • Move the "Change" button from each individual question to the section
  • Will live on a Pizza for a while to be used for next UR session

image

@DafyddLlyr
Copy link
Contributor Author

On the result page, move 'Change' button to be by the section, rather than by the question

@DafyddLlyr DafyddLlyr added the UAT (User Agent Testing) actively being tested before deployment label Oct 11, 2023
@github-actions
Copy link

github-actions bot commented Oct 11, 2023 •
edited
Loading

Removed vultr server and associated DNS entries

@DafyddLlyr DafyddLlyr requested a review from a team October 11, 2023 17:20
@DafyddLlyr DafyddLlyr marked this pull request as ready for review October 11, 2023 18:39
@DafyddLlyr DafyddLlyr force-pushed the dp/move-change-to-section branch from 9f3e35b to c0529eb Compare October 30, 2023 13:36
@DafyddLlyr
Copy link
Contributor Author

Feedback from UR -
image

@DafyddLlyr DafyddLlyr closed this Nov 29, 2023
@ianjon3s ianjon3s mentioned this pull request Dec 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ianjon3s ianjon3s ianjon3s approved these changes

Assignees
No one assigned
Labels
UAT (User Agent Testing) actively being tested before deployment
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DafyddLlyr @ianjon3s