-
Notifications
You must be signed in to change notification settings - Fork 1.5k
feat: add access control to project #2687
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
Changes from 26 commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
619d8b3
feat: add access control to project
Kitenite 2822a2c
update invite email
Kitenite 55be678
update invite email
Kitenite 8f11028
accept invitation
Kitenite fde910f
update invite check
Kitenite 0eb23cd
working restrictions
Kitenite 40e4cbd
redirect auth
Kitenite a994c17
allow login
Kitenite 4ce60f0
handle auth
Kitenite 2b1523b
Merge branch 'main' into feat/restrict-project-access
Kitenite fa90775
Merge branch 'main' into feat/restrict-project-access
Kitenite d25ceab
Update apps/web/client/src/app/auth/callback/route.ts
Kitenite 8631951
fix design mockup
Kitenite 0229bf2
fix typecheck
Kitenite 680ac81
fix features page
Kitenite b4ea748
debug auth
Kitenite c7ab278
fix redirect create
Kitenite af48643
working auth
Kitenite 434ac88
save progress
Kitenite 8dcefa9
refactor return url
Kitenite 1b006d5
get invitation without token
Kitenite d62fc10
handle submit error
Kitenite 3e0a300
remove urljoin
Kitenite f015310
update to suggestions
Kitenite d0a5714
update origin
Kitenite a0c9e13
clean up
Kitenite dc75913
handle copy
Kitenite ab7dd18
update copy
Kitenite df07cef
clean up
Kitenite 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
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
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
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
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.
💡 Verification agent
🧩 Analysis chain
Broader ignore pattern for preload script LGTM; ensure previously tracked files are untracked
Good call generalizing to onlook-preload-script.*. If any such files were already tracked, remove them from the index so the ignore takes effect.
🏁 Script executed:
Length of output: 478
🏁 Script executed:
Length of output: 282
Remove tracked preload script so the new ignore pattern takes effect
The file
apps/web/client/public/onlook-preload-script.js
is still in the index, so your.gitignore
won’t ignore future changes. Please untrack it:• Tracked file:
Run:
and commit the removal.
🤖 Prompt for AI Agents