Skip to content
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

fix: improved team upgrade screen to also show unpublished teams #12492

Merged
merged 6 commits into from
Nov 29, 2023

Conversation

PeerRich
Copy link
Member

@PeerRich PeerRich commented Nov 22, 2023

todo:

  • figure out hasUnpublishedTeam

before:

  • even if you have an unpublished team, you see the upgrade screen:

CleanShot 2023-11-22 at 13 29 12@2x

after:
CleanShot 2023-11-22 at 14 03 05@2x

@PeerRich PeerRich linked an issue Nov 22, 2023 that may be closed by this pull request
Copy link

linear bot commented Nov 22, 2023

CAL-2743 show unpublished teams

right now, if you have an unpublished team, it still shows the empty screen

Copy link

vercel bot commented Nov 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 29, 2023 9:44pm
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 29, 2023 9:44pm
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 29, 2023 9:44pm
4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Nov 29, 2023 9:44pm
cal-demo ⬜️ Ignored (Inspect) Visit Preview Nov 29, 2023 9:44pm
qa ⬜️ Ignored (Inspect) Visit Preview Nov 29, 2023 9:44pm
ui ⬜️ Ignored (Inspect) Visit Preview Nov 29, 2023 9:44pm

@github-actions github-actions bot added Low priority Created by Linear-GitHub Sync teams area: teams, round robin, collective, managed event-types 🐛 bug Something isn't working 🙋🏻‍♂️help wanted Help from the community is appreciated labels Nov 22, 2023
Copy link
Contributor

github-actions bot commented Nov 22, 2023

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link to collect XP and win prizes!

@zomars zomars added the core area: core, team members only label Nov 22, 2023
Copy link
Contributor

github-actions bot commented Nov 22, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

Copy link

deploysentinel bot commented Nov 22, 2023

Current Playwright Test Results Summary

✅ 347 Passing - ⚠️ 10 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 11/29/2023 09:43:46pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 49720da

Started: 11/29/2023 09:36:47pm UTC

⚠️ Flakes

📄   apps/web/playwright/booking/selectQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Phone Question and Each Other Question Booking With Select Question and checkbox group Question Select required and checkbox group required
Retry 1Initial Attempt
0.33% (1) 1 / 306 run
failed over last 7 days
2.61% (8) 8 / 306 runs
flaked over last 7 days

📄   apps/web/playwright/event-types.e2e.ts • 2 Flakes

Top 1 Common Error Messages

null

2 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Event Types tests user -- future Different Locations Tests can select 'display on booking page' option when multiple organizer input type are present
Retry 1Initial Attempt
0% (0) 0 / 268 runs
failed over last 7 days
4.48% (12) 12 / 268 runs
flaked over last 7 days
Event Types tests user -- legacy Different Locations Tests can add single organizer address location without display location public option
Retry 1Initial Attempt
0% (0) 0 / 268 runs
failed over last 7 days
0.75% (2) 2 / 268 runs
flaked over last 7 days

📄   apps/web/playwright/booking/addressQuestione2e/addressQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Address Question and Each Other Question Booking With Address Question and Multi email Question Address and Multi email not required
Retry 1Initial Attempt
0% (0) 0 / 315 runs
failed over last 7 days
0.63% (2) 2 / 315 runs
flaked over last 7 days

📄   apps/web/playwright/organization/organization-invitation.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Organization Invitation (non verified)
Retry 1Initial Attempt
9.78% (31) 31 / 317 runs
failed over last 7 days
21.77% (69) 69 / 317 runs
flaked over last 7 days

📄   apps/web/playwright/team/team-invitation.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Team Invitation (non verified)
Retry 1Initial Attempt
2.36% (7) 7 / 297 runs
failed over last 7 days
17.85% (53) 53 / 297 runs
flaked over last 7 days

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 2 Flakes

Top 1 Common Error Messages

null

2 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 1Initial Attempt
6.29% (21) 21 / 334 runs
failed over last 7 days
90.42% (302) 302 / 334 runs
flaked over last 7 days
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 1Initial Attempt
0.89% (3) 3 / 336 runs
failed over last 7 days
74.70% (251) 251 / 336 runs
flaked over last 7 days

📄   apps/web/playwright/organization/organization-creation.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Organization should be able to create an organization and complete onboarding
Retry 1Initial Attempt
2.52% (8) 8 / 318 runs
failed over last 7 days
94.65% (301) 301 / 318 runs
flaked over last 7 days

📄   apps/web/playwright/booking/multipleEmailQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Multiple Email Question and Each Other Question Booking With Multiple Email Question and Address Question Multiple Email required and Address required
Retry 1Initial Attempt
2.47% (8) 8 / 324 runs
failed over last 7 days
1.23% (4) 4 / 324 runs
flaked over last 7 days

View Detailed Build Results


@PeerRich PeerRich requested a review from zomars November 23, 2023 21:31
@PeerRich PeerRich added Medium priority Created by Linear-GitHub Sync and removed Low priority Created by Linear-GitHub Sync labels Nov 23, 2023
Copy link
Member

Choose a reason for hiding this comment

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

Reduced team router boilerplate

zomars
zomars previously approved these changes Nov 29, 2023
@zomars zomars merged commit 7f23ae1 into main Nov 29, 2023
37 checks passed
@zomars zomars deleted the 12468-cal-2743-show-unpublished-teams branch November 29, 2023 21:48
jakazzy pushed a commit to jakazzy/cal.com that referenced this pull request Dec 5, 2023
…com#12492)

* improved team upgrade screen to also show unpublished teams

* Update TeamsListing.tsx

* bunch of stuff

---------

Co-authored-by: Omar López <zomars@me.com>
hbjORbj pushed a commit to codemod-com/cal.com-demo that referenced this pull request Dec 21, 2023
…com#12492)

* improved team upgrade screen to also show unpublished teams

* Update TeamsListing.tsx

* bunch of stuff

---------

Co-authored-by: Omar López <zomars@me.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working core area: core, team members only 🙋🏻‍♂️help wanted Help from the community is appreciated Medium priority Created by Linear-GitHub Sync teams area: teams, round robin, collective, managed event-types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CAL-2743] show unpublished teams
2 participants