Conversation
Member
|
bugbot run |
anekkanti
approved these changes
Mar 24, 2026
Member
|
@billrich2001 Thank you for the PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What was changed
Add optional fields to codec_server for custom error message and link
Why?
Spurred by customer ticket; Match functionality with UI.
Checklist
Closes [Feature Request] codec_server custom error message and link #408
How was this tested:
Added unit tests, and ran against existing tests and CI
Any docs updates needed?
Auto generated with changes
Note
Medium Risk
Moderate risk because it extends the
codec_serverschema and request/response mapping for namespaces, which can affect plan/state and API payloads, but changes are additive and covered by new unit/acceptance tests.Overview
Adds optional
codec_server.custom_error_messageandcodec_server.custom_error_linkto thetemporalcloud_namespaceresource and exposes the same fields as computed outputs on thetemporalcloud_namespace(s)data sources.Updates the namespace spec/state mapping to read/write these fields to the Cloud API (
CustomErrorMessage.Default.{Message,Link}), and expands test coverage (acceptance + unit) to validate null vs set behaviors and import stability. Docs are regenerated to reflect the new attributes.Written by Cursor Bugbot for commit 203e4e3. This will update automatically on new commits. Configure here.