feat: add APPSMITH_HIDE_HELP_BUTTON env var to hide help/support icon#41925
feat: add APPSMITH_HIDE_HELP_BUTTON env var to hide help/support icon#41925tomjose92 wants to merge 1 commit into
Conversation
Introduces a runtime config flag that, when enabled, hides the help/support button (the "?" menu containing documentation, chat, send-support-info and what's-new) both on the applications home page and inside the app editor. - ce/configs: thread `hideHelpButton` through INJECTED_CONFIGS, getConfigsFromEnvVars and getAppsmithConfigs (mirrors the existing disableIframeWidgetSandbox pattern) - public/index.html: inject APPSMITH_HIDE_HELP_BUTTON into window.APPSMITH_FEATURE_CONFIGS for runtime (no rebuild) configuration - HomepageHeaderAction / HelpButton: gate render on the flag - EnvVariables.java: register the env var so it is manageable server-side - docker.env.sh, helm values/README/snapshot, jest.config: defaults set to false so existing installs are unaffected Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01E72oeKwZBo84Wemo4TXrd2
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (10)
WalkthroughAdds ChangesHide help button wiring
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
Adds a runtime configuration flag,
APPSMITH_HIDE_HELP_BUTTON, that hides the help/support button — the?menu containing Documentation, Send support info, Chat with us, and What's new — in both places it appears:/applications) —HomepageHeaderAction.tsxHelpButton.tsxIt follows the existing
APPSMITH_DISABLE_IFRAME_WIDGET_SANDBOXpattern, so it is injected at runtime viawindow.APPSMITH_FEATURE_CONFIGSand requires no client rebuild — operators just set the env var on a self-hosted (Docker/Helm) instance.Usage
Defaults to
false, so existing installs are unaffected.Changes
ce/configs/index.ts,types.ts— threadhideHelpButtonthroughINJECTED_CONFIGS,getConfigsFromEnvVars()andgetAppsmithConfigs()public/index.html— injectAPPSMITH_HIDE_HELP_BUTTONintowindow.APPSMITH_FEATURE_CONFIGSHomepageHeaderAction.tsx— gate the help menu with!hideHelpButtonHelpButton.tsx— earlyreturn nullwhen the flag is set (after all hooks)EnvVariables.java— register the env var so it is recognized/manageable server-sidedocker.env.sh,helm/values.yaml,helm/README.md, helm snapshot,jest.config.js— defaults set tofalseWhy it's regression-safe
falseeverywhere → behavior is identical for existing installs; the change is purely additive.!isAirgappedInstance && !hideHelpButton).Notes
false).Testing
APPSMITH_HIDE_HELP_BUTTON=true→ help icon hidden on home page and in editorfalse→ help icon visible as before🤖 Generated with Claude Code
Generated by Claude Code
Summary by CodeRabbit
New Features
Bug Fixes
Chores