Skip to content

chore: [app-router-migration 37] migrate connect and join page#13205

Merged
PeerRich merged 2 commits into
calcom:mainfrom
codemod:migrate/connect-and-join
Jan 16, 2024
Merged

chore: [app-router-migration 37] migrate connect and join page#13205
PeerRich merged 2 commits into
calcom:mainfrom
codemod:migrate/connect-and-join

Conversation

@hbjORbj
Copy link
Copy Markdown
Contributor

@hbjORbj hbjORbj commented Jan 12, 2024

What does this PR do?

Fixes # (issue)

Requirement/Documentation

  • If there is a requirement document, please, share it here.
  • If there is ab UI/UX design document, please, share it here.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Tests (Unit/Integration/E2E or any other test)
  • This change requires a documentation update

How should this be tested?

  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my PR needs changes to the documentation
  • I haven't checked if my changes generate no new warnings
  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 12, 2024

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

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 12, 2024

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

@hbjORbj hbjORbj changed the title chore: [app-router-migration 37] migrate more page chore: [app-router-migration 37] migrate connect and join page Jan 12, 2024
@keithwillcode keithwillcode added High priority Created by Linear-GitHub Sync community Created by Linear-GitHub Sync labels Jan 12, 2024
@keithwillcode keithwillcode self-assigned this Jan 12, 2024
@keithwillcode keithwillcode self-requested a review January 12, 2024 18:34
@PeerRich PeerRich merged commit b09f3e5 into calcom:main Jan 16, 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 High priority Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants