Skip to content

feat: update help center url links - #178

Open
vivekSonata wants to merge 3 commits into
masterfrom
vivekP/ENT-12170
Open

feat: update help center url links#178
vivekSonata wants to merge 3 commits into
masterfrom
vivekP/ENT-12170

Conversation

@vivekSonata

@vivekSonata vivekSonata commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

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=0

Changes Impemented:-
Help center Links updated with new URLs for these mentioned below:

  • HELP_CENTER_LINK
  • HELP_CENTER_BLACKBOARD
  • HELP_CENTER_CANVAS
  • HELP_CENTER_CORNERSTONE
  • HELP_CENTER_DEGREED
  • HELP_CENTER_MOODLE
  • HELP_CENTER_SAP
  • HELP_CENTER_SAML_LINK
  • HELP_CENTER_GROUPS_INVITE_LINK
  • Note: Config-level surfaces (ENTERPRISE_SUPPORT_URL, ENTERPRISE_SUPPORT_LEARNER_CREDIT_URL, LEARNER_SUPPORT_URL) audited and covered in edx-internal#14791

Attached Screenshots:
Admin Portal:-
image
image
image
image
image
image

Learner Portal:-
image
image
image
image

Copilot AI lite review requested due to automatic review settings August 11, 2026 11:38
@vivekSonata
vivekSonata requested review from a team as code owners August 11, 2026 11:38

Copilot AI left a comment

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.

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_LINK to point to the new enterprise-support.edx.org help center.

High severity issue identified during review:

  • HELP_CENTER_GROUPS_INVITE_LINK now 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.

Comment thread src/components/settings/data/constants.js
@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.95%. Comparing base (61ac2ba) to head (0b88d9c).
⚠️ Report is 1 commits behind head on master.

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI review requested due to automatic review settings August 13, 2026 08:47

Copilot AI left a comment

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.

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 ';

Copilot AI review requested due to automatic review settings August 13, 2026 11:04

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI left a comment

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.

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/';

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.

3 participants