Skip to content

fix: add QUOTE_NOTIFICATIONS to SimplifiedNotification TypeEnum - #59

Closed
tutkat wants to merge 2 commits into
mainfrom
hotfix/simplifiednotification-quote-notifications
Closed

tutkat wants to merge 2 commits into
mainfrom
hotfix/simplifiednotification-quote-notifications

Conversation

@tutkat

@tutkat tutkat commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • UAT API started returning QUOTE_NOTIFICATIONS as a value for SimplifiedNotification.type
  • The value was missing from the generated TypeEnum, causing IllegalArgumentException during deserialization
  • This is a temporary hotfix until PR feat: Fabric API Sync 4.30 to 4.31 by GitHub Action for @tutkat #58 (sync 4.30→4.31) is merged — that PR contains the full regenerated SDK with this value included

Root Cause

The sync workflow fetched the spec at a moment when docs.equinix.com had not yet published QUOTE_NOTIFICATIONS, but the UAT API was already returning it. The spec has since been updated.

Test plan

  • CI tests on main pass after merge (ConnectionsApiTest should no longer throw Unexpected value 'QUOTE_NOTIFICATIONS')

@tutkat
tutkat requested a review from a team as a code owner August 17, 2026 12:44
@tutkat

tutkat commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Closing — fix will be included in sync PR #58 instead.

@tutkat tutkat closed this Aug 17, 2026
@tutkat
tutkat deleted the hotfix/simplifiednotification-quote-notifications branch August 17, 2026 13:12
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.

1 participant