-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
Organization system #335
Merged
riderx
merged 105 commits into
Cap-go:development
from
WcaleNieWolny:organization-system
Dec 2, 2023
Merged
Organization system #335
riderx
merged 105 commits into
Cap-go:development
from
WcaleNieWolny:organization-system
Dec 2, 2023
Conversation
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
1 task
Updated dependencies detected. Learn more about Socket for GitHub ↗︎
|
DO NOT UPDATE PLAYWRTIGHT, I AM USING OLDER VERSION ON PURPOSE TO AVOID THIS ISSUE |
/linkpr Cap-go/CLI#138 |
Kudos, SonarCloud Quality Gate passed! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR implements V0 for the organization system. This is ALMOST ready, however due to the size of this PR rigorous testing have to be performed to make sure after merging this PR everything will go smoothly.
I am opening this to get feedback from @riderx but there are a couple things missing from this PR as is:
Videos showing changesRigorous testing, I have tested this code, however there are likely some bugs that will be cough with more testsSQL queries to migrate production (So that this can be merged)Clear oftypes.ts
, there are a lot of unnecessary changes there/claim #258
Summary by CodeRabbit
DropdownOrganization.vue
for organization selection and invitation management.redirect
property to theTab
interface for optional redirection.InfoRow.vue
by removing unnecessary code.SharedUserTable.vue
.Account.vue
and moved it to a new service modulephotos.ts
.useSupabase
function to infer the type ofoptions
variable.main
function inupload_link/index.ts
to retrieve user ID based on API key and app ID.