-
Notifications
You must be signed in to change notification settings - Fork 238
[comp] Production Deploy #1877
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[comp] Production Deploy #1877
Conversation
* feat(organization): add transfer ownership functionality and do visible the section del organization only for owner * feat(organization): implement transfer ownership endpoint and update related schemas * fix(organization): throw BadRequestException for missing user ID in transfer ownership --------- Co-authored-by: Tofik Hasanov <annexcies@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
PR SummaryAdds organization ownership transfer (backend + UI) and updates auditor job to use Groq model and Firecrawl v2, with OpenAPI docs updated.
Written by Cursor Bugbot for commit d50d0d3. This will update automatically on new commits. Configure here. |
|
|
Graphite Automations"Auto-assign PRs to Author" took an action on this PR • (12/08/25)1 reviewer was added to this PR based on Mariano Fuentes's automation. |
| params: Promise<{ orgId: string }>; | ||
| }) { | ||
| const { orgId } = await params; | ||
| console.log('[OrganizationSettings Debug] orgId:', orgId); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Debug console.log statement left in production code
A debug console.log('[OrganizationSettings Debug] orgId:', orgId) statement was left in the production code. This appears to be debugging output that was accidentally committed and will log organization IDs to the server console on every settings page load.
| // Get current user's member record | ||
| const currentUserMember = await db.member.findFirst({ | ||
| where: { organizationId, userId: currentUserId }, | ||
| }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Deactivated owner can still transfer ownership
The query for currentUserMember does not filter by deactivated: false, but the newOwnerMember query does. This inconsistency could allow a deactivated member who still has the owner role to transfer ownership, even though they are supposed to be inactive. Adding deactivated: false to the currentUserMember query would make the checks consistent.
|
🎉 This PR is included in version 1.70.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.