Repository navigation
fix(admin): set poc to owner email when creating organization - #2037
rohilsurana wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Add owner email to creation payload web/sdk/admin/views/organizations/list/create.tsx |
The submission payload now includes metadata.poc, set to the lowercased owner email. |
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Bug fix
Possibly related PRs
- raystack/frontier#2019: Updates organization editing to preserve existing metadata, including
metadata.poc.
Suggested reviewers: rohanchkrabrty
Merge Risk: 
Minimal · up to 9efae
-
The new owner-email metadata has no established merge-blocking issue. Merge after normal checks; older organizations still require the stated backfill.
Pre-merge checks: 
2
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Coverage Report for CI Build 41395913700Coverage remained the same at 55.992%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
Summary
The admin "Create organization" panel saved only
countryin the org metadata. So orgs created there had nopoc(point of contact email).This change saves the owner email as
pocwhen the panel creates an org. The email is lowercased, so it matches the email stored on the owner's user record.Effects
metadata.pocset to the owner email.metadata.poc. It will now show the owner email for these orgs, instead of an empty cell.pocstays after later edits.Not in this PR
poc. They need a one time backfill.Testing