Skip to content

Fix NoMethodError on unexpected response from USPS API#8513

Merged
tomas-nava merged 1 commit intomainfrom
tomas/fix-no-method-error-on-unexpected-request-enroll-response
May 31, 2023
Merged

Fix NoMethodError on unexpected response from USPS API#8513
tomas-nava merged 1 commit intomainfrom
tomas/fix-no-method-error-on-unexpected-request-enroll-response

Conversation

@tomas-nava
Copy link
Contributor

🛠 Summary of changes

We encountered a case where a NoMethodError was being raised upon receiving a response from USPS API in an unexpected format. The error came from a case where we were using [variable].class.class_name instead of [variable].class.name.

class_name is a method provided by the yard gem that apparently is not available in production.

@tomas-nava tomas-nava requested review from a team and NavaTim May 31, 2023 01:57
changelog: Bug Fixes, In-person proofing, Fix NoMethodError on unexpected response from USPS
@tomas-nava tomas-nava force-pushed the tomas/fix-no-method-error-on-unexpected-request-enroll-response branch from aaa6d43 to a436fff Compare May 31, 2023 02:04
Copy link
Contributor

@JackRyan1989 JackRyan1989 left a comment

Choose a reason for hiding this comment

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

LGTM!

@tomas-nava tomas-nava merged commit d871ac7 into main May 31, 2023
@tomas-nava tomas-nava deleted the tomas/fix-no-method-error-on-unexpected-request-enroll-response branch May 31, 2023 16:19
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.

4 participants