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

fix(modals): consolidate error modal, welcome modal, and update modal #4942

Merged

Conversation

nathan-power
Copy link
Contributor

What this PR does 📖

Which issue(s) this PR fixes 🔨

Special notes for reviewers 🗒️

Additional comments 🎤

@nathan-power nathan-power self-assigned this Sep 20, 2022
@nathan-power nathan-power linked an issue Sep 20, 2022 that may be closed by this pull request
@github-actions github-actions bot added the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label Sep 20, 2022
@josephmcg josephmcg self-requested a review September 20, 2022 03:51
@josephmcg josephmcg merged commit 063800f into dev Sep 20, 2022
@josephmcg josephmcg deleted the 4726-consolidate-error-modal-welcome-modal-and-update-modal branch September 20, 2022 03:56
@github-actions github-actions bot removed Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA bug labels Sep 20, 2022
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.

long content in update modal breaks layout consolidate error modal, welcome modal, and update modal
2 participants