Skip to content

fix: host entry not created when user added to team in org#17927

Merged
emrysal merged 2 commits into
calcom:mainfrom
kart1ka:fix/host-entry-not-created-when-user-added-to-team
Dec 2, 2024
Merged

fix: host entry not created when user added to team in org#17927
emrysal merged 2 commits into
calcom:mainfrom
kart1ka:fix/host-entry-not-created-when-user-added-to-team

Conversation

@kart1ka
Copy link
Copy Markdown
Collaborator

@kart1ka kart1ka commented Dec 1, 2024 •
edited
Loading

What does this PR do?

https://www.loom.com/share/d4081b0c4caf498f8be1ed0727e055e0?sid=4217387e-8546-479a-a1a5-ad948f0b2ea0

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

@graphite-app graphite-app Bot added the community Created by Linear-GitHub Sync label Dec 1, 2024
@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 1, 2024

@kart1ka is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@graphite-app graphite-app Bot requested a review from a team December 1, 2024 08:08
@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 labels Dec 1, 2024
@dosubot dosubot Bot added the organizations area: organizations, orgs label Dec 1, 2024
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Dec 1, 2024 •
edited
Loading

Graphite Automations

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

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

"Add community label" took an action on this PR • (12/01/24)

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

"Add ready-for-e2e label" took an action on this PR • (12/01/24)

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

@Praashh Praashh self-assigned this Dec 1, 2024
Praashh
Praashh approved these changes Dec 1, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@Praashh Praashh left a comment

Choose a reason for hiding this comment

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

LGTM!!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2024 •
edited
Loading

E2E results are ready!

@emrysal emrysal enabled auto-merge (squash) December 2, 2024 18:59
@emrysal emrysal merged commit c524a3c into calcom:main Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Praashh Praashh Praashh approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@Praashh Praashh

Labels

🐛 bug Something isn't working community Created by Linear-GitHub Sync Low priority Created by Linear-GitHub Sync organizations area: organizations, orgs ready-for-e2e teams area: teams, round robin, collective, managed event-types

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[CAL-4732] Host entry not created when user added to team

3 participants

@kart1ka @Praashh @emrysal