Skip to content

PM-5495: Remove Wipro payment section from challenge details#7227

Merged
jmgasper merged 1 commit into
developfrom
PM-5495
Jul 1, 2026
Merged

PM-5495: Remove Wipro payment section from challenge details#7227
jmgasper merged 1 commit into
developfrom
PM-5495

Conversation

@jmgasper

@jmgasper jmgasper commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

What was broken
Wipro and Topgear group challenge detail pages rendered an extra Payments section with Wipro payroll and WCP payment terms.

Root cause
The challenge detail specification component appended a hard-coded Payments article whenever the challenge belonged to the Wipro community group.

What was changed
Removed the Wipro-only Payments article from the challenge detail specification while keeping the Wipro community detection for existing sidebar behavior.

Any added/updated tests
Added a challenge detail specification regression test that renders a Wipro group challenge and verifies the payment section copy is absent.

Validation

  • npm run jest -- tests/shared/components/challenge-detail/Specification/index.jsx
  • npm run lint
  • npm test
  • npm run build

What was broken
Wipro and Topgear group challenge detail pages rendered an extra Payments section with Wipro payroll and WCP payment terms.

Root cause
The challenge detail specification component appended a hard-coded Payments article whenever the challenge belonged to the Wipro community group.

What was changed
Removed the Wipro-only Payments article from the challenge detail specification while keeping the Wipro community detection for existing sidebar behavior.

Any added/updated tests
Added a challenge detail specification regression test that renders a Wipro group challenge and verifies the payment section copy is absent.
@jmgasper jmgasper requested a review from kkartunov as a code owner July 1, 2026 06:36
@jmgasper jmgasper merged commit 6fb960d into develop Jul 1, 2026
7 checks passed
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.

1 participant