Skip to content

fix registration config disable - #177

Merged
goshacodes merged 1 commit into
mainfrom
feat/user-registration
Aug 15, 2026
Merged

fix registration config disable#177
goshacodes merged 1 commit into
mainfrom
feat/user-registration

Conversation

@goshacodes

Copy link
Copy Markdown
Member

No description provided.

@augmentcode

augmentcode Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor
🤖 Augment PR Summary

Summary: This PR fixes clearing a client registration configuration through the configuration API.

  • Migrates affected Central configuration request handlers from zio-schema JSON decoding to zio-json codec decoding.
  • This preserves the custom Option[Patch[A]] contract, where an explicit JSON null means delete rather than no change.
  • Updates client creation and update handling along with other configuration mutation endpoints for consistent request decoding.
  • Adds an integration-style controller test proving registrationFlow: null is forwarded as Some(Patch.Deleted).
Technical Notes: The change relies on each request model's derived JsonCodec and keeps the existing service-layer validation and response behavior unchanged.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@goshacodes
goshacodes merged commit b5880ac into main Aug 15, 2026
6 checks passed
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