chore: add dev SUPPORT_URL config for help button verification#16
Merged
Merged
Conversation
Sets commonAppConfig.SUPPORT_URL and an instructor-dashboard per-app SUPPORT_URL override in site.config.dev.tsx so the help button can be verified locally without standing up a backend. site.config.build.tsx is intentionally not modified — these are example URLs for local dev and shouldn't ship in production builds. Operators configure SUPPORT_URL through whatever mechanism suits their deployment. Refs openedx/frontend-base#245 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
brian-smith-tcril
force-pushed
the
help-button
branch
from
d7439ba to
55e10b3
Compare
arbrandes
approved these changes
Apr 29, 2026
jesusbalderramawgu
approved these changes
Apr 29, 2026
diana-villalvazo-wgu
approved these changes
Apr 29, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Summary
Companion PR to openedx/frontend-base#251. Adds
commonAppConfig.SUPPORT_URLand an instructor-dashboard per-app override insite.config.dev.tsxso the help button can be verified locally without standing up a backend.site.config.build.tsxis intentionally not modified — these are example URLs for local dev and shouldn't ship in production builds.This PR is independent of the others — it's just dev-config scaffolding. Useful to merge whenever, since the dev experience without it is "configure your own URLs to verify."
Test plan
make dev-packages— dev site comes uphttps://example.com/help/https://example.com/help/instructor(per-app override wins over commonAppConfig)commonAppConfig.SUPPORT_URLand the instructor override → Help link disappears on both routes🤖 Generated with Claude Code