-
Notifications
You must be signed in to change notification settings - Fork 120
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
fix: personal spaces can be made private without a pro plan #776
fix: personal spaces can be made private without a pro plan #776
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @BlankParticle and the rest of your teammates on Graphite |
@BlankParticle i think we should change this to never allow personal spaces to be public in the first place |
there might be use cases for people to have open personal spaces, also we need to alllow people who have accidently made their spaces open |
maybe only a tiny few usecases if personal space is true, allow it to be set to private, but dont allow to be set to open (open is disabled) |
ok, will do that |
6da7062
to
28de097
Compare
@McPizza0, have done as you asked, also added the missing backend checks |
apps/web/src/app/[orgShortcode]/[spaceShortcode]/settings/page.tsx
Outdated
Show resolved
Hide resolved
28de097
to
9b56a6b
Compare
Merge activity
|
9b56a6b
to
e33c521
Compare
What does this PR do?
Fixes ENG-140
Personal spaces can always be made private regardless of pro plan
Once affected spaces are private again, we don't allow to open them to avoid confusion
Type of change
Checklist
Required
console.logs
git pull origin main
Appreciated