-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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: New Organization Schema along with the ability to invite existing users to an organization #13002
Merged
Merged
feat: New Organization Schema along with the ability to invite existing users to an organization #13002
Changes from all commits
Commits
Show all changes
70 commits
Select commit
Hold shift + click to select a range
2e03d9a
Add profile table
hariombalhara 04b91da
wip
hariombalhara 367ede5
Get switcher working
hariombalhara cb94c67
Add organization seeder
hariombalhara 442d430
wip
hariombalhara 3e06263
Remove dead code
hariombalhara 6ae0c24
Fix unit tests
hariombalhara deb6e03
relevantProfile -> profile
hariombalhara 0a2d343
cache bust
hariombalhara c5010e9
One more comment
hariombalhara 82529e0
Squash migrations
hariombalhara 8a38cf9
Copy to profile table
hariombalhara 99ba771
Update schema
hariombalhara 7f69117
Fix existing user invite flow
hariombalhara cf7273d
Add profile table
hariombalhara 03d9bae
Copy to profile table
hariombalhara 11d4c6c
Update schema
hariombalhara dca50b6
Format
hariombalhara 04c40f3
Add profile table
hariombalhara abe419c
Format
hariombalhara e119472
wip
hariombalhara 2fc8cc0
wip
hariombalhara 7cebdc1
Add profile relation everywhere
hariombalhara ffbd8a0
Merge branch '01-13-New_profile_table_and_data_copy_to_it' into organ…
hariombalhara 40dc7bf
Fix schema
hariombalhara ade3a02
Merge remote-tracking branch 'origin/organizationId-column' into 01-1…
hariombalhara bf296f9
Merge branch '01-13-New_profile_table_and_data_copy_to_it' into organ…
hariombalhara c58ded3
Merge branch 'organizationId-column' into 01-15-feat_Add_relations_wi…
hariombalhara bf9d7ac
Fix invite handler
hariombalhara 16bf486
Merge branch '01-15-feat_Add_relations_with_profile_wherever_needed' …
hariombalhara 63f56ff
Add profileId during eventTypec reation
hariombalhara 3267566
Get dashboard working with profile relatioon
hariombalhara 50d1e33
Add workflow create repository
hariombalhara 0f823e2
Add profileId to host
hariombalhara 179f7af
Add profileId to credential table
hariombalhara d3cc068
Add destinationCalendar repository
hariombalhara a3cd4b8
Add profileId to membership
hariombalhara 09774a5
Revert all profile related changes
hariombalhara 5d74142
Add acme and dunder
hariombalhara 1a9b3f9
Fix membership profileId usgae
hariombalhara 44c755f
Various fixes
hariombalhara 6dea2f1
Some renaming
hariombalhara 6d41df2
Fix failing tests
hariombalhara ccdf507
Debugging next-aut
hariombalhara bd4c04c
Merge remote-tracking branch 'origin/main' into organizationId-column
hariombalhara fd8c3f9
Consolidate migrations
hariombalhara a4bfb8e
Testing fixes
hariombalhara b559b30
Fix unit tests
hariombalhara 1f39f5a
Merge remote-tracking branch 'origin/main' into organizationId-column
hariombalhara 1688ca4
Self review feedback
hariombalhara ebb1abb
Merge remote-tracking branch 'origin/main' into organizationId-column
hariombalhara c12a9c8
PR feedback
hariombalhara cb9b3be
Merge remote-tracking branch 'origin/main' into organizationId-column
hariombalhara 1a86f2f
Merge remote-tracking branch 'origin/main' into organizationId-column
hariombalhara 2729778
Fix orgMigration as well
hariombalhara 423ee4f
Add profile schema and copy users to it
hariombalhara 7cc3153
Remove migrations from here
hariombalhara 9dd96f2
Fix type issues
hariombalhara 2c777e9
Merge remote-tracking branch 'origin/01-13-New_profile_table_and_data…
hariombalhara b781030
Merge remote-tracking branch 'origin/main' into organizationId-column
hariombalhara 4783b48
Fix type error after main merge
hariombalhara 9d657f2
Remove movedFromUserId
hariombalhara 82018db
Merge remote-tracking branch 'origin/main' into organizationId-column
hariombalhara f387434
Add missing migration file
hariombalhara 1247de4
Self review addressed
hariombalhara 72cea47
Merge branch 'main' into organizationId-column
joeauyeung 07bc2eb
Joe's review addressed
hariombalhara c626a8e
Fix profile switcher
hariombalhara 8ffb0a1
Merge branch 'main' into organizationId-column
joeauyeung f74ba88
Merge branch 'main' into organizationId-column
hariombalhara 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 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 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 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 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 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 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 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 |
---|---|---|
|
@@ -729,17 +729,17 @@ const EventTypeListHeading = ({ | |
return ( | ||
<div className="mb-4 flex items-center space-x-2"> | ||
<Avatar | ||
alt={profile?.name || ""} | ||
alt={profile.name || ""} | ||
href={teamId ? `/settings/teams/${teamId}/profile` : "/settings/my-account/profile"} | ||
imageSrc={`${bookerUrl}${teamId ? "/team" : ""}/${profile.slug}/avatar.png`} | ||
imageSrc={profile.image} | ||
size="md" | ||
className="mt-1 inline-flex justify-center" | ||
/> | ||
<div> | ||
<Link | ||
href={teamId ? `/settings/teams/${teamId}/profile` : "/settings/my-account/profile"} | ||
className="text-emphasis font-bold"> | ||
{profile?.name || ""} | ||
{profile.name || ""} | ||
</Link> | ||
{membershipCount && teamId && ( | ||
<span className="text-subtle relative -top-px me-2 ms-2 text-xs"> | ||
|
@@ -751,13 +751,13 @@ const EventTypeListHeading = ({ | |
</Link> | ||
</span> | ||
)} | ||
{profile?.slug && ( | ||
{profile.slug && ( | ||
<Link href={`${bookerUrl}/${profile.slug}`} className="text-subtle block text-xs"> | ||
{`${bookerUrl.replace("https://", "").replace("http://", "")}/${profile.slug}`} | ||
</Link> | ||
)} | ||
</div> | ||
{!profile?.slug && !!teamId && ( | ||
{!profile.slug && !!teamId && ( | ||
<button onClick={() => publishTeamMutation.mutate({ teamId })}> | ||
<Badge variant="gray" className="-ml-2 mb-1"> | ||
{t("upgrade")} | ||
|
@@ -816,7 +816,7 @@ const CTA = ({ data }: { data: GetByViewerResponse }) => { | |
const Actions = () => { | ||
return ( | ||
<div className="hidden items-center md:flex"> | ||
<TeamsFilter popoverTriggerClassNames="mb-0" showVerticalDivider={true} /> | ||
<TeamsFilter useProfileFilter popoverTriggerClassNames="mb-0" showVerticalDivider={true} /> | ||
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. It is a way to opt-in to profile based DB querying. As event-type supports profileId now. |
||
</div> | ||
); | ||
}; | ||
|
@@ -853,7 +853,6 @@ const Main = ({ | |
}) => { | ||
const isMobile = useMediaQuery("(max-width: 768px)"); | ||
const searchParams = useCompatSearchParams(); | ||
const orgBranding = useOrgBranding(); | ||
|
||
if (!data || status === "pending") { | ||
return <SkeletonLoader />; | ||
|
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.
User already has profile now which has the organization