Skip to content

Namespace exists use downcase check#1248

Merged
HaiHui886 merged 1 commit into
mainfrom
sync_mr_2705_540e6c7
Jul 9, 2026
Merged

Namespace exists use downcase check#1248
HaiHui886 merged 1 commit into
mainfrom
sync_mr_2705_540e6c7

Conversation

@Rader

@Rader Rader commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary:

  • Fixes an issue where organizations with different casing (e.g., "qwen" vs "Qwen") were treated as distinct entities.
  • Implements case-insensitive namespace validation by converting paths to lowercase before existence checks to prevent duplicate organization creation.

@Rader

Rader commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator Author

Depends on #1247

@Rader Rader mentioned this pull request Jul 9, 2026
Merged
HaiHui886
HaiHui886 approved these changes Jul 9, 2026
View reviewed changes

@HaiHui886 HaiHui886 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@HaiHui886
HaiHui886 merged commit 9823cb8 into main Jul 9, 2026
7 checks passed
@HaiHui886
HaiHui886 deleted the sync_mr_2705_540e6c7 branch July 9, 2026 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HaiHui886 HaiHui886 HaiHui886 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Rader @HaiHui886 @pulltheflower