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

feat: Add step in team creation form for selecting event type and create a team event #15286

Merged
merged 31 commits into from
Sep 5, 2024

Conversation

anikdhabal
Copy link
Contributor

@anikdhabal anikdhabal commented May 31, 2024

What does this PR do?

Screenshot 2024-06-01 035049

Result:-

Screenshot 2024-06-01 035227

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected)
  • N/A - I have added a Docs issue here if this PR makes changes that would require a documentation change
  • I have added or modified automated tests that prove my fix is effective or that my feature works (PRs might be rejected if logical changes are not properly tested)

Sorry, something went wrong.

Copy link

vercel bot commented May 31, 2024

@anikdhabal 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 added the community Created by Linear-GitHub Sync label May 31, 2024
@graphite-app graphite-app bot requested a review from a team May 31, 2024 22:21
@github-actions github-actions bot added consumer event-types area: event types, event-types Medium priority Created by Linear-GitHub Sync ✨ feature New feature or request 🧹 Improvements Improvements to existing features. Mostly UX/UI labels May 31, 2024
Copy link
Contributor

github-actions bot commented May 31, 2024

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

@dosubot dosubot bot added the teams area: teams, round robin, collective, managed event-types label May 31, 2024
@dosubot dosubot bot added this to the v4.2 milestone May 31, 2024
Copy link

graphite-app bot commented May 31, 2024

Graphite Automations

"Add community label" took an action on this PR • (05/31/24)

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

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

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

@anikdhabal anikdhabal changed the title feat: Create team event on a new team creation feat: Add step in team creation form for selecting event type and create a team event May 31, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Copy link
Member

@SomayChauhan SomayChauhan left a comment

Choose a reason for hiding this comment

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

@anikdhabal
Copy link
Contributor Author

https://www.loom.com/share/2e468d97ba6a44c8a23f0eff0df02cf2?sid=e5d25316-5c68-47d1-918b-1604af8d458b

@anikdhabal could you add fix this otherwise looks good 💯

Thanks for the review. I've fixed that.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
SomayChauhan
SomayChauhan previously approved these changes Jun 11, 2024
Copy link
Member

@SomayChauhan SomayChauhan left a comment

Choose a reason for hiding this comment

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

Tested, LGTM!!
Thankyou @anikdhabal, Great Work

@keithwillcode keithwillcode added this pull request to the merge queue Jun 11, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 11, 2024
@keithwillcode keithwillcode added this pull request to the merge queue Jun 11, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 11, 2024
Copy link
Member

@CarinaWolli CarinaWolli left a comment

Choose a reason for hiding this comment

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

some e2e tests are failing because of the new step when creating a team

@anikdhabal
Copy link
Contributor Author

some e2e tests are failing because of the new step when creating a team

Yeah, I am on it.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Copy link
Contributor

github-actions bot commented Aug 16, 2024

E2E results are ready!

@anikdhabal anikdhabal marked this pull request as ready for review August 16, 2024 07:27

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@dosubot dosubot bot removed this from the v4.4 milestone Aug 19, 2024
@keithwillcode keithwillcode modified the milestones: v4.4, v4.5 Aug 19, 2024
Copy link
Member

@CarinaWolli CarinaWolli left a comment

Choose a reason for hiding this comment

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

I think we should allow the user to skip the form, as they might not yet know which event type they need.

I am also wondering if it would also make sense to already assign team members. It might be too much, but then the event type would be functional (cc @ciaranha)

anikdhabal and others added 5 commits August 29, 2024 16:26

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@anikdhabal anikdhabal requested a review from Udit-takkar August 30, 2024 09:12
Udit-takkar and others added 5 commits August 31, 2024 12:22

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@CarinaWolli CarinaWolli dismissed hariombalhara’s stale review September 5, 2024 16:31

all requested changes were addressed

Copy link
Member

@CarinaWolli CarinaWolli left a comment

Choose a reason for hiding this comment

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

Nice work, the changes look good to me and looks like all other feedback has been addressed 🙏

@CarinaWolli CarinaWolli merged commit fced9a6 into calcom:main Sep 5, 2024
32 of 39 checks passed
@dosubot dosubot bot modified the milestones: v4.5, v4.6 Sep 15, 2024
@keithwillcode keithwillcode modified the milestones: v4.6, v4.5 Sep 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Created by Linear-GitHub Sync community-interns The team responsible for reviewing, testing and shipping low/medium community PRs consumer event-types area: event types, event-types ✨ feature New feature or request 🧹 Improvements Improvements to existing features. Mostly UX/UI Medium priority Created by Linear-GitHub Sync ready-for-e2e teams area: teams, round robin, collective, managed event-types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CAL-2960] default event-types after creating a team
7 participants