Skip to content

Conversation

@mrotondo
Copy link
Contributor

Link to pivotal/JIRA issue

Is PM acceptance required? (delete one)

  • No - merge after code review approval

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

What was done?

  • This updates our GYR-Efiler version to one that is using the MeF SDK v16, unless we're in the production environment AND it's still 2025.

How to test?

  • I've manually tested this by running Efile::GyrEfilerService.run_efiler_command("test", "acks", "fake_submission_id"). Before this change, I get an error saying that the version of the WSDL is too low, and after this change, I get an error saying that the submission ID is invalid (which is correct)

/HTTP transport error: javax.net.ssl.SSLException/,
]

# TODO: Update this to unconditionally return the newer commit hash (8c46c9) after Jan 1 2026
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👉 👈 can you put this in jira instead of a todo comment

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@github-actions
Copy link

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

@mrotondo mrotondo merged commit 9da806b into main Oct 24, 2025
5 of 8 checks passed
@mrotondo mrotondo deleted the TEF-80-update-me-f-sdk-to-v-16 branch October 24, 2025 22:51
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