Skip to content

refactor(BA-7749): switch login_client_type update DTO to Unset - #14406

Draft
wngh4628 wants to merge 5 commits into
mainfrom
feature/BA-7749-login-client-type-unset
Draft

refactor(BA-7749): switch login_client_type update DTO to Unset#14406
wngh4628 wants to merge 5 commits into
mainfrom
feature/BA-7749-login-client-type-unset

Conversation

@wngh4628

@wngh4628 wngh4628 commented Sep 8, 2026

Copy link
Copy Markdown

Switch the login_client_type update DTO from the legacy Sentinel marker to Unset (omitted = no change, null = clear).

  • common/dto/manager/v2/login_client_type/request.py: UpdateLoginClientTypeInput.description changes from str | Sentinel | None = SENTINEL to str | None | Unset = UNSET; the field description now reads "Omit to leave unchanged; null clears."
  • manager/api/adapters/login_client_type/adapter.py: the Sentinel branch for description is replaced with TriState.from_unset(input.description).

🤖 Generated with Claude Code

https://claude.ai/code/session_017idp6rKVV33XrzJG1yWYXo


📚 Documentation preview 📚: https://sorna--14406.org.readthedocs.build/en/14406/


📚 Documentation preview 📚: https://sorna-ko--14406.org.readthedocs.build/ko/14406/

@wngh4628
wngh4628 requested a review from a team as a code owner September 8, 2026 06:16
@github-actions github-actions Bot added size:S 10~30 LoC comp:manager Related to Manager component comp:common Related to Common component area:docs Documentations labels Sep 8, 2026
@wngh4628
wngh4628 force-pushed the feature/BA-7749-login-client-type-unset branch from 9028cc5 to c529cbc Compare September 8, 2026 07:21
wngh4628 and others added 2 commits September 8, 2026 07:23
Co-authored-by: octodog <mu001@lablup.com>
@github-actions github-actions Bot added size:L 100~500 LoC comp:client Related to Client component comp:cli Related to CLI component and removed size:S 10~30 LoC labels Sep 9, 2026
wngh4628 and others added 2 commits September 9, 2026 10:04
Co-authored-by: octodog <mu001@lablup.com>
@wngh4628
wngh4628 marked this pull request as draft September 9, 2026 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs Documentations comp:cli Related to CLI component comp:client Related to Client component comp:common Related to Common component comp:manager Related to Manager component size:L 100~500 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant