Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: Improve add users to org#25314

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
volnei merged 4 commits intomainfromfix/improve_add_user_to_org
Nov 21, 2025
Merged

Conversation

@volnei
Copy link
Contributor

@volneivolnei commentedNov 21, 2025
edited
Loading

What does this PR do?

User assignment improvements

@keithwillcodekeithwillcode added corearea: core, team members only foundation labelsNov 21, 2025
@volneivolnei marked this pull request as ready for reviewNovember 21, 2025 13:47
@graphite-appgraphite-appbot requested a review froma teamNovember 21, 2025 13:47
@vercel
Copy link

vercelbot commentedNov 21, 2025
edited
Loading

The latest updates on your projects. Learn more aboutVercel for GitHub.

2 Skipped Deployments
ProjectDeploymentPreviewCommentsUpdated (UTC)
calIgnoredIgnoredNov 21, 2025 2:42pm
cal-euIgnoredIgnoredNov 21, 2025 2:42pm

…k prevention- Add tests for cross-tenant hijack prevention security fix- Test that users belonging to different organizations are blocked- Test that users with null organizationId are blocked (prevents legacy user hijacking)- Test that users belonging to correct organization succeed- Test new user creation flow- Test user activation/deactivation flows- Test custom attribute syncing- All 8 tests passing locally with proper type safety (no 'as any' usage)Co-Authored-By: Volnei Munhoz <volnei.munhoz@gmail.com>
keithwillcode
keithwillcode previously approved these changesNov 21, 2025
@volneivolneienabled auto-merge (squash)November 21, 2025 14:07
Copy link
Contributor

@cubic-dev-aicubic-dev-aibot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Copy link
Contributor

@cubic-dev-aicubic-dev-aibot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@volneivolnei merged commit5cb1d4a intomainNov 21, 2025
39 checks passed
@volneivolnei deleted the fix/improve_add_user_to_org branchNovember 21, 2025 15:18
@github-actions
Copy link
Contributor

E2E results are ready!

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@cubic-dev-aicubic-dev-ai[bot]cubic-dev-ai[bot] left review comments

@keithwillcodekeithwillcodekeithwillcode approved these changes

Assignees

No one assigned

Labels

corearea: core, team members onlyfoundationready-for-e2esize/L

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@volnei@keithwillcode

[8]ページ先頭

©2009-2025 Movatter.jp