-
Notifications
You must be signed in to change notification settings - Fork 1
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
INT-2298: chore: Migrate "/settings/organizations/*" pages #146
Conversation
Current dependencies on/for this PR: This stack of pull requests is managed by Graphite. |
Hey there and thank you for opening this pull request! 👋🏼 We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted. Details:
|
); | ||
|
||
const LayoutWrapper = (page: React.ReactElement) => ( | ||
<> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this react fragment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addressed
0ddbb1e
to
73b23d9
Compare
This PR is being marked as stale due to inactivity. |
73b23d9
to
39a8fd5
Compare
* Make private URLs easier to copy-paste from web app * Apply suggestions from code review * Update apps/web/pages/event-types/index.tsx * lint --------- Co-authored-by: Peer Richelsen <[email protected]> Co-authored-by: Peer Richelsen <[email protected]>
Co-authored-by: Omar López <[email protected]>
…ure (calcom#12988) * make no-meeting-found page use ssr * remove-route-groups * add LayoutHOC * fix * fix * ensure proper types for withLayout function --------- Co-authored-by: Greg Pabian <[email protected]>
86b61e1
to
e22f852
Compare
What does this PR do?
Fixes # (issue)
Requirement/Documentation
Type of change
How should this be tested?
Mandatory Tasks
Checklist