-
Notifications
You must be signed in to change notification settings - Fork 11.6k
feat: ooo-v2 #13621
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
Merged
Merged
feat: ooo-v2 #13621
Changes from all commits
Commits
Show all changes
76 commits
Select commit
Hold shift + click to select a range
6b68712
first commit, changes in getSlots endpoint to deliver ooo data
alannnc 63ef710
Removing old features handle type and user
alannnc 0237870
merge with main
alannnc 618aea8
remove unnecesary import
alannnc 167c0d7
Remove old files from away or booking redirection
alannnc 1d4847e
Merge branch 'main' of github.com:calcom/cal.com into ooo-v2
alannnc 8e5f50f
wip
alannnc 87658f7
Merge branch 'main' of github.com:calcom/cal.com into ooo-v2
alannnc 748b20f
merge with remote
alannnc 6972b48
wip
alannnc 093b73a
changed trpc response type and removing no forward ooo days from avai…
alannnc ee59e90
Merge branch 'main' of github.com:calcom/cal.com into ooo-v2
alannnc e0edd25
redo of out of office page, still wip
alannnc def3134
Merge branch 'main' of github.com:calcom/cal.com into ooo-v2
alannnc 9ad098d
reasons in ooo entry
alannnc edea4ed
Merge branch 'main' of github.com:calcom/cal.com into ooo-v2
alannnc 2dd017e
clean up and add missing translations
alannnc 16aee20
fix text translation components
alannnc 69c421c
solve merge with remote
alannnc 2f8cd2f
Add reasons table and default options and implement it on booker
alannnc 0e5f300
fixes
alannnc 2bef090
Merge branch 'main' into ooo-v2
alannnc f08ea69
return away from ssr
alannnc 2514e83
Merge branch 'ooo-v2' of github.com:alannnc/cal.com into ooo-v2
alannnc 5bbfb67
return away from ssr
alannnc 90209d8
fix type in test
alannnc 9f6ad2f
Merge branch 'main' into pr/13621
zomars 59860d6
Merge branch 'ooo-v2' of github.com:alannnc/cal.com into ooo-v2
alannnc fddda72
bg-muted + new field notes ooo + query param to open modal
alannnc 1356213
minor text fixes anc tw class
alannnc 107367d
Fix styles and tw classes
alannnc 43fca31
Merge branch 'main' of github.com:calcom/cal.com into ooo-v2
alannnc ba9616e
Merge branch 'main' of github.com:calcom/cal.com into ooo-v2
alannnc 927d9e5
fix orgs getting url params in app router
alannnc 7b4199d
remove org from booker store as not needed
alannnc 77f0ff2
redo test for ooo
alannnc 1d835b6
Merge branch 'main' into ooo-v2
alannnc ceb81b5
Update apps/web/pages/settings/my-account/out-of-office/index.tsx
PeerRich fe04a87
merge with remote
alannnc 8863b41
attack review changes, styles for ooo
alannnc bc775b1
Merge branch 'ooo-v2' of github.com:alannnc/cal.com into ooo-v2
alannnc 0a7408b
merge with remote
alannnc cae9969
fixing timezone issues with ooo slots
alannnc 5d49954
merge with main
alannnc 426235d
improve performance to calculate slots for empty column_view
alannnc 9144bd0
fix types
alannnc a830cda
fix types
alannnc 613c1a0
fix types
alannnc a78eb53
Merge branch 'main' into ooo-v2
12876e8
Merge branch 'main' into ooo-v2
alannnc 31cbbed
Merge branch 'main' into ooo-v2
alannnc 6730173
Merge branch 'ooo-v2' of github.com:alannnc/cal.com into ooo-v2
alannnc 005c84f
fix ooo without to user
alannnc 9ae3619
fix ooo in slots component render conditionals
alannnc 9ee16c8
fix offset top for ooo in slots
alannnc a5b7caf
filter out ooo days in column view with no forwarding
alannnc 09ffa74
add more margin to ooo in slots
alannnc c04f8ec
fix flatten function on irregular availability
alannnc 981c497
Merge branch 'main' of github.com:calcom/cal.com into ooo-v2
alannnc ff98d89
Merge branch 'main' into ooo-v2
emrysal 286394e
fix types
alannnc a15edb6
Merge branch 'ooo-v2' of github.com:alannnc/cal.com into ooo-v2
alannnc ed386ca
Merge branch 'main' into ooo-v2
alannnc 9c5e177
Merge branch 'main' into ooo-v2
alannnc df5833f
Merge branch 'main' into pr/13621
zomars 9384c45
Update index.tsx
zomars 394b802
Merge branch 'main' into ooo-v2
alannnc 142bb2c
Merge branch 'main' into ooo-v2
PeerRich 1cf4ff3
Merge branch 'main' into ooo-v2
PeerRich 3653255
Merge branch 'main' into ooo-v2
CarinaWolli 5ec48e7
Merge branch 'main' into ooo-v2
CarinaWolli fedf00b
Merge branch 'main' into ooo-v2
CarinaWolli a2fca52
Merge branch 'main' into ooo-v2
PeerRich 5e5bd1f
Merge branch 'main' into ooo-v2
zomars edf8380
Update OutOfOfficeInSlots.tsx
zomars e4a12ae
Merge branch 'main' into ooo-v2
alannnc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -87,25 +87,6 @@ export function UserPage(props: InferGetServerSidePropsType<typeof getServerSide | |
| isEmbed ? "border-booker border-booker-width bg-default rounded-md border" : "", | ||
| "max-w-3xl px-4 py-24" | ||
| )}> | ||
| {isRedirect && ( | ||
| <div className="mb-8 rounded-md bg-blue-100 p-4 dark:border dark:bg-transparent"> | ||
| <h2 className="text-default mb-2 text-sm font-semibold dark:text-white"> | ||
| {t("user_redirect_title", { | ||
| username: fromUserNameRedirected, | ||
| })}{" "} | ||
| 🏝️ | ||
| </h2> | ||
| <p className="text-default text-sm"> | ||
| {t("user_redirect_description", { | ||
| profile: { | ||
| username: user.username, | ||
| }, | ||
| username: fromUserNameRedirected, | ||
| })}{" "} | ||
| 😄 | ||
| </p> | ||
| </div> | ||
| )} | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No longer needed. |
||
| <div className="mb-8 text-center"> | ||
| <UserAvatar | ||
| size="xl" | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,7 +2,6 @@ import { type GetServerSidePropsContext } from "next"; | |
| import { z } from "zod"; | ||
|
|
||
| import { getServerSession } from "@calcom/features/auth/lib/getServerSession"; | ||
| import { handleTypeRedirection } from "@calcom/features/booking-redirect/handle-type"; | ||
| import { getBookingForReschedule, getBookingForSeatedEvent } from "@calcom/features/bookings/lib/get-booking"; | ||
| import type { GetBookingType } from "@calcom/features/bookings/lib/get-booking"; | ||
| import { orgDomainConfig } from "@calcom/features/ee/organizations/lib/orgDomains"; | ||
|
|
@@ -105,7 +104,7 @@ async function getUserPageProps(context: GetServerSidePropsContext) { | |
| const username = usernames[0]; | ||
| const { rescheduleUid, bookingUid } = context.query; | ||
| const { currentOrgDomain, isValidOrgDomain } = orgDomainConfig(context.req, context.params?.orgSlug); | ||
| let outOfOffice = false; | ||
|
|
||
| const isOrgContext = currentOrgDomain && isValidOrgDomain; | ||
| if (!isOrgContext) { | ||
| const redirect = await getTemporaryOrgRedirect({ | ||
|
|
@@ -132,18 +131,6 @@ async function getUserPageProps(context: GetServerSidePropsContext) { | |
| notFound: true, | ||
| } as const; | ||
| } | ||
| // If user is found, quickly verify bookingRedirects | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Feature no longer needed. |
||
| const result = await handleTypeRedirection({ | ||
| userId: user.id, | ||
| username, | ||
| slug, | ||
| }); | ||
| if (result && result.outOfOffice) { | ||
| outOfOffice = true; | ||
| } | ||
| if (result && result.redirect?.destination) { | ||
| return result; | ||
| } | ||
|
|
||
| let booking: GetBookingType | null = null; | ||
| if (rescheduleUid) { | ||
|
|
@@ -176,9 +163,9 @@ async function getUserPageProps(context: GetServerSidePropsContext) { | |
| length: eventData.length, | ||
| metadata: eventData.metadata, | ||
| }, | ||
| away: outOfOffice, | ||
| user: username, | ||
| slug, | ||
| away: false, | ||
| trpcState: ssr.dehydrate(), | ||
| isBrandingHidden: user?.hideBranding, | ||
| isSEOIndexable: user?.allowSEOIndexing, | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Changes on this file are to remove previous feature of profile forwarding, no longer needed.