-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
feat: More admin options for organizations #12424
feat: More admin options for organizations #12424
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
|
Current dependencies on/for this PR:
This stack of pull requests is managed by Graphite. |
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link. |
a7451b9
to
0b23a4a
Compare
0b23a4a
to
5e7e22a
Compare
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 New Page AddedThe following page was added to the bundle from the code in this PR:
One Page Changed SizeThe following page changed size from the code in this PR compared to its base branch:
DetailsOnly the gzipped size is provided here based on an expert tip. First Load is the size of the global bundle plus the bundle for the individual page. If a user were to show up to your website and land on a given page, the first load size represents the amount of javascript that user would need to download. If Any third party scripts you have added directly to your app using the The "Budget %" column shows what percentage of your performance budget the First Load total takes up. For example, if your budget was 100kb, and a given page's first load size was 10kb, it would be 10% of your budget. You can also see how much this has increased or decreased compared to the base branch of your PR. If this percentage has increased by 20% or more, there will be a red status indicator applied, indicating that special attention should be given to this. If you see "+/- <0.01%" it means that there was a change in bundle size, but it is a trivial enough amount that it can be ignored. |
Current Playwright Test Results Summary✅ 318 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 11/21/2023 05:28:57am UTC) Run DetailsRunning Workflow PR Update on Github Actions Commit: ee48a8c Started: 11/21/2023 05:20:20am UTC
|
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
2FA Tests should allow a user to enable 2FA and login using 2FA
Retry 1 • Initial Attempt |
0.41% (1)1 / 245 runfailed over last 7 days |
32.65% (80)80 / 245 runsflaked over last 7 days |
📄 apps/web/playwright/booking/checkboxGroupQuestion.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Booking With Checkbox Group Question and Each Other Question Booking With Checkbox Group Question and Address Question Booking With Checkbox Group Question and Phone Question Checkbox Group required and Phone required
Retry 1 • Initial Attempt |
0% (0)0 / 243 runsfailed over last 7 days |
1.23% (3)3 / 243 runsflaked over last 7 days |
📄 packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 2 Flakes
Top 1 Common Error Messages
|
2 Test Cases Affected |
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe according to system theme when no theme is configured through Embed API
Retry 1 • Initial Attempt |
2.97% (8)8 / 269 runsfailed over last 7 days |
57.62% (155)155 / 269 runsflaked over last 7 days |
Popup Tests should be able to reschedule
Retry 2 • Retry 1 • Initial Attempt |
20.45% (55)55 / 269 runsfailed over last 7 days |
74.72% (201)201 / 269 runsflaked over last 7 days |
📄 apps/web/playwright/managed-event-types.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Managed Event Types tests Can create managed event type
Retry 1 • Initial Attempt |
13.55% (34)34 / 251 runsfailed over last 7 days |
13.55% (34)34 / 251 runsflaked over last 7 days |
5e7e22a
to
13afd95
Compare
13afd95
to
ee48a8c
Compare
ee48a8c
to
856ed7a
Compare
3ed696a
to
5e30855
Compare
packages/trpc/server/routers/viewer/organizations/adminDelete.handler.ts
Show resolved
Hide resolved
packages/trpc/server/routers/viewer/organizations/adminUpdate.handler.ts
Outdated
Show resolved
Hide resolved
5e30855
to
7a36a0b
Compare
Thanks for the review @ThyMinimalDev. Addressed all comments. Also, FYI it's a stacked PR. So, I would really appreciate if you could review the target branch PR as well as that should be merged first. |
e732246
to
9c0baa9
Compare
@ThyMinimalDev No longer a stacked PR now. It's targeted to main now. If you could do a final review that would be great. |
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.
looking good
What does this PR do?
Stacked over #12358. Please merge that first.
Admin Options for Organizations
Loom Demo
Other fixes
Type of change
How should this be tested?
Mandatory Tasks