feat: update help center url links - #178
Conversation
There was a problem hiding this comment.
Pull request overview
Updates a settings-module constant that drives “Help Center” links in the Learner Credit Management (budget member invite) UI.
Changes:
- Updated
HELP_CENTER_GROUPS_INVITE_LINKto point to the newenterprise-support.edx.orghelp center.
High severity issue identified during review:
HELP_CENTER_GROUPS_INVITE_LINKnow targets the generic/s/landing page even though the UI text is “Help Center: Invite/Inviting Budget Members”, so users may no longer land on the intended help content.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #178 +/- ##
=======================================
Coverage 87.95% 87.95%
=======================================
Files 842 842
Lines 20877 20877
Branches 4404 4406 +2
=======================================
Hits 18363 18363
Misses 2452 2452
Partials 62 62 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
src/components/settings/data/constants.js:35
- The HELP_CENTER_SAP URL string has a trailing space at the end (
.../success-factors). That space will be part of the href and can cause the link to 404 or be percent-encoded unexpectedly.
export const HELP_CENTER_SAP = 'https://enterprise-support.edx.org/s/topic/0TORc000000GC5FOAW/success-factors ';
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
src/components/settings/data/constants.js:41
- Two HELP_CENTER_* constants in this block still point to the legacy Zendesk domain (business-support.edx.org): HELP_CENTER_SAP_IDP_LINK and HELP_CENTER_BRANDING_LINK. Given the PR’s stated goal of updating hardcoded Help Center URLs to enterprise-support.edx.org, these look like missed migrations and may leave users routed to the deprecated site.
export const HELP_CENTER_SAML_LINK = 'https://enterprise-support.edx.org/s/topic/0TORc000000GC3dOAG/single-signon-sso';
export const HELP_CENTER_SAP_IDP_LINK = 'https://business-support.edx.org/hc/en-us/articles/360005205314';
export const HELP_CENTER_BRANDING_LINK = 'https://business-support.edx.org/hc/en-us/sections/8739219372183';
export const HELP_CENTER_GROUPS_INVITE_LINK = 'https://enterprise-support.edx.org/s/';
JIRA Work item: https://2u-internal.atlassian.net/browse/ENT-12170
Context:-
Support has migrated the help center from Zendesk (business-support.edx.org) to Salesforce (enterprise-support.edx.org). This PR updates the hardcoded help center URL literals in frontend-app-admin-portal's source code. It complements companion PR edx-internal#14791, which updates the config-driven URLs (ENTERPRISE_SUPPORT_URL, ENTERPRISE_SUPPORT_LEARNER_CREDIT_URL) served via static MFE config and MFE_CONFIG_OVERRIDES. Together the two PRs cover all admin-portal help center surfaces referenced in
https://docs.google.com/spreadsheets/d/1CBPtUWfqLgSofpvHtZZ0ONRs8pg2mUthSykdMoUAy_I/edit?gid=0#gid=0Changes Impemented:-
Help center Links updated with new URLs for these mentioned below:
Attached Screenshots:






Admin Portal:-
Learner Portal:-



