Skip to content

refactor: environment-gate in-dev features instead of team-member whi… - #2286

Open
MayankMittal1 wants to merge 5 commits into
masterfrom
chore/env-gate-dev-features
Open

refactor: environment-gate in-dev features instead of team-member whi…#2286
MayankMittal1 wants to merge 5 commits into
masterfrom
chore/env-gate-dev-features

Conversation

@MayankMittal1

Copy link
Copy Markdown
Contributor

…telist

Replace useIsTestUser()/TEST_FEATURES_ALLOW_LIST with isDevFeatureEnabled (!isProduction). In-dev features (mobile-first UI, Stocks, GoodDollar, bank transfer, coming-soon vaults) are now visible on qa/preview builds and hidden from everyone in production, removing the team-member exception.

…telist

Replace useIsTestUser()/TEST_FEATURES_ALLOW_LIST with isDevFeatureEnabled
(!isProduction). In-dev features (mobile-first UI, Stocks, GoodDollar,
bank transfer, coming-soon vaults) are now visible on qa/preview builds
and hidden from everyone in production, removing the team-member exception.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@cursor

cursor Bot commented Jul 22, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-app Ignored Ignored Preview Jul 22, 2026 6:55pm
solid-app-staging Ignored Ignored Preview Jul 22, 2026 6:55pm

Request Review

@claude

claude Bot commented Jul 22, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.


Reviewed with Claude Code

Comment thread lib/config.ts
export const EXPO_PUBLIC_COINGECKO_API_KEY = process.env.EXPO_PUBLIC_COINGECKO_API_KEY ?? '';
export const isProduction = EXPO_PUBLIC_ENVIRONMENT === 'production';
// In-development features: visible on qa/preview builds, hidden in production.
export const isDevFeatureEnabled = !isProduction;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MayankMittal1 remove unused code

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.

2 participants