Skip to content
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

🚀 [Feature] Disable new signups through server controlled feature flags #1922

Merged

Conversation

AmruthPillai
Copy link
Owner

Fixes #1592

This PR contains changes in the environment variables to move them from VITE client side feature flags which were only set during build time, to access them from the server and pass them down to the client so they can be modified on-the-fly.

The two new feature flags are:

DISABLE_SIGNUPS=true|false
DISABLE_EMAIL_AUTH=true|false

Setting DISABLE_SIGNUPS to true would make the Register page display this banner:

Screenshot 2024-05-29 at 10 34 12 AM

Setting DISABLE_EMAIL_AUTH to true would make the Login page display this banner:

Screenshot 2024-05-29 at 10 34 18 AM


Also, the flag SKIP_STORAGE_BUCKET_CHECK has been renamed to STORAGE_SKIP_BUCKET_CHECK, to clearly indicate that it's not a feature flag but just a server configuration that should not be exposed to the client.

@AmruthPillai AmruthPillai linked an issue May 29, 2024 that may be closed by this pull request
1 task
Copy link

nx-cloud bot commented May 29, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit bf498b5. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

@AmruthPillai AmruthPillai merged commit 3f29ca3 into main May 29, 2024
4 checks passed
@AmruthPillai AmruthPillai deleted the 1592-feature-the-possibility-to-disable-new-sign-ups branch May 29, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] The possibility to disable new sign ups.
1 participant