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

Revert NJ 287 - commit 90c1bd7acd3bf6ff32b5193de5e092104b2e87ac #5625

Merged
merged 1 commit into from
Feb 21, 2025

Conversation

jachan
Copy link
Contributor

@jachan jachan commented Feb 21, 2025

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?

  • Seeing errors in XML, attempting to revert 28b logic to fix it

How to test?

Screenshots (for visual changes)

  • Before
  • After

@jachan jachan changed the title Revert commit 90c1bd7acd3bf6ff32b5193de5e092104b2e87ac Revert NJ 287 - commit 90c1bd7acd3bf6ff32b5193de5e092104b2e87ac Feb 21, 2025
Copy link

Heroku app: https://gyr-review-app-5625-4b689a4df6f6.herokuapp.com/
View logs: heroku logs --app gyr-review-app-5625 (optionally add --tail)

@jachan jachan merged commit f968463 into main Feb 21, 2025
7 checks passed
@jachan jachan deleted the revert-nj-287 branch February 21, 2025 21:49
jachan added a commit that referenced this pull request Feb 24, 2025
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.

2 participants