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

Feature/aten nys 28 edit profile tab updates #51

Merged
merged 6 commits into from
Nov 27, 2023

Conversation

nathanielwoodland
Copy link
Contributor

No description provided.

@routinet
Copy link
Contributor

Functional testing passed no problem. I'm suggesting we take the time to refresh the touched .theme file areas wrt coding standards.

@nathanielwoodland
Copy link
Contributor Author

Hi @routinet, as discussed, I've brought the surrounding conditional logic up appropriate coding standards. Please re-review. Thanks!

@kzalewski kzalewski merged commit d73438a into main Nov 27, 2023
1 check passed
@kzalewski kzalewski deleted the feature/aten-nys-28--edit-profile-tab-updates branch November 27, 2023 18:53
@routinet routinet added the ready Tested and ready for deployment (unless on-hold) label Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Tested and ready for deployment (unless on-hold)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants