Skip to content

feat: v2 managed user create default schedule given timeZone#14727

Merged
pumfleet merged 6 commits intomainfrom
managed-user-default-schedule
Apr 26, 2024
Merged

feat: v2 managed user create default schedule given timeZone#14727
pumfleet merged 6 commits intomainfrom
managed-user-default-schedule

Conversation

@supalarry
Copy link
Copy Markdown
Contributor

What

When creating a managed OAuth user - if timeZone is given, create default schedule and have "user.defaultScheduleId" as part of response when creating managed user.

Why

To simplify creation of default schedules - users now already have a schedule created and they can update it if they want.

How

  1. Add new function to schedules service to create default user schedule in schedules service: function to create default user schedule
  2. Use it when creating managed user + e2e tests in managed user: create default schedule if timeZone provided

@supalarry supalarry requested a review from a team April 24, 2024 10:10
@graphite-app graphite-app bot requested review from a team and removed request for a team April 24, 2024 10:10
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 24, 2024

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

@supalarry supalarry removed the request for review from a team April 24, 2024 10:10
@keithwillcode keithwillcode added core area: core, team members only platform Anything related to our platform plan labels Apr 24, 2024
@supalarry supalarry requested a review from a team April 24, 2024 10:10
@dosubot dosubot bot added api area: API, enterprise API, access token, OAuth automated-tests area: unit tests, e2e tests, playwright ✨ feature New feature or request labels Apr 24, 2024
@graphite-app
Copy link
Copy Markdown

graphite-app bot commented Apr 24, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (04/24/24)

1 reviewer was added to this PR based on Keith Williams's automation.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 24, 2024

📦 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! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel bot commented Apr 24, 2024

No failed tests 🎉

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 24, 2024

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 Apr 26, 2024 8:35am
platform-starter-kit ❌ Failed (Inspect) Apr 26, 2024 8:35am
2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Apr 26, 2024 8:35am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Apr 26, 2024 8:35am

@pumfleet pumfleet disabled auto-merge April 24, 2024 17:12
@pumfleet pumfleet enabled auto-merge (squash) April 24, 2024 17:13
@pumfleet pumfleet merged commit d004815 into main Apr 26, 2024
@pumfleet pumfleet deleted the managed-user-default-schedule branch April 26, 2024 09:20
p6l-richard pushed a commit to p6l-richard/cal.com-fork that referenced this pull request Jul 22, 2024
…14727)

* schedules service: function to create default user schedule

* managed user: create default schedule if timeZone provided

* swagger update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api area: API, enterprise API, access token, OAuth automated-tests area: unit tests, e2e tests, playwright core area: core, team members only ✨ feature New feature or request platform Anything related to our platform plan

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants