Skip to content

feat: add FAQ links to warning messages for merchant troubleshooting#1268

Merged
papallavi merged 4 commits intodevelopfrom
DTCRCMERC-3133
Feb 19, 2026
Merged

feat: add FAQ links to warning messages for merchant troubleshooting#1268
papallavi merged 4 commits intodevelopfrom
DTCRCMERC-3133

Conversation

@papallavi
Copy link
Copy Markdown
Contributor

@papallavi papallavi commented Feb 2, 2026

Description

Add help_url field to all warning messages directing merchants to troubleshooting documentation. Enhances console output to display FAQ links for common issues like message hidden, invalid options, and integration errors.
https://paypal.atlassian.net/browse/DTCRCMERC-3133

Screenshots

Testing instructions

Add help_url field to all warning messages directing merchants to troubleshooting documentation. Enhances console output to display FAQ links for common issues like message hidden, invalid options, and integration errors.
@hemra-1 hemra-1 requested a review from yeul February 3, 2026 15:51
Comment thread src/utils/logger.js Outdated
Comment thread src/utils/faq.js Outdated
Comment thread src/library/zoid/message/component.js Outdated
@Braluna-pp
Copy link
Copy Markdown
Collaborator

In /messages/interface.test.js consider doing the following,

Creating tests that seert help_url for warnings.

Reason: current tests only check selector/location and won’t catch regressions to the new help_url behavior. Add assertions for help_url (and optionally appended description) in warning tests.

- Remove description mutation in logger to avoid breaking telemetry parsers
- Normalize FAQ base URL before concatenation to prevent double slashes
- Add help_url to unsafe lander warning for consistency
- Add comprehensive test coverage for help_url in warnings:
  * Created dedicated test suite for help_url in message interface
  * Added help_url assertions to validation tests
  * Created unit tests for getFaqUrl function with 9 test cases
  * Tests verify URL normalization and fallback behavior
Copy link
Copy Markdown
Collaborator

@Braluna-pp Braluna-pp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Comment thread src/utils/faq.js Outdated
Comment thread globals.js Outdated
- Remove duplicate FAQ_PATHS entries (INVALID_SELECTOR, NOT_IN_DOCUMENT, UNSAFE_LANDER)
- Consolidate to shared paths: INTEGRATION and INVALID_OPTIONS
- Remove unnecessary env.FAQ_BASE_URL check in globals.js
- Update all source files to use consolidated path names
- Update tests to reflect consolidated paths
@papallavi papallavi requested review from Braluna-pp and yeul February 6, 2026 04:42
Comment thread globals.js Outdated
Comment thread src/utils/faq.js Outdated
@papallavi papallavi merged commit 3a7d6b5 into develop Feb 19, 2026
69 checks passed
@papallavi papallavi deleted the DTCRCMERC-3133 branch February 19, 2026 05:38
github-actions Bot pushed a commit that referenced this pull request Feb 24, 2026
## [1.78.0](v1.77.1...v1.78.0) (2026-02-24)

### Features

* add FAQ links to warning messages for merchant troubleshooting ([#1268](#1268)) ([3a7d6b5](3a7d6b5))
* Add language_requested field to fpti ([#1249](#1249)) ([2866ae0](2866ae0))
* add locale param ([#1255](#1255)) ([df4bde6](df4bde6))

### Documentation

* add subagents section and update persona (DTCRCMERC-4677) ([dcf9202](dcf9202))
* add v5/v6 service dependency map for agents ([bf44bb2](bf44bb2))
* align crcoffersserv abbreviation to CRCO in AGENTS diagram ([068ec29](068ec29))
* expand subagents guidance (DTCRCMERC-4677) ([845449f](845449f))
* normalize deep-dive dependency wording ([37a2bd0](37a2bd0))
* remove deep internal service details from public AGENTS ([a60e77a](a60e77a))
* strengthen workflow orchestration in AGENTS.md (DTCRCMERC-4677) ([b2ddef2](b2ddef2))
* update PR template and agent docs (DTCRCMERC-4677) ([9689af5](9689af5))
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.78.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

lyerabati16 pushed a commit that referenced this pull request Apr 8, 2026
## [1.78.0](v1.77.1...v1.78.0) (2026-02-24)

### Features

* add FAQ links to warning messages for merchant troubleshooting ([#1268](#1268)) ([3a7d6b5](3a7d6b5))
* Add language_requested field to fpti ([#1249](#1249)) ([2866ae0](2866ae0))
* add locale param ([#1255](#1255)) ([df4bde6](df4bde6))

### Documentation

* add subagents section and update persona (DTCRCMERC-4677) ([dcf9202](dcf9202))
* add v5/v6 service dependency map for agents ([bf44bb2](bf44bb2))
* align crcoffersserv abbreviation to CRCO in AGENTS diagram ([068ec29](068ec29))
* expand subagents guidance (DTCRCMERC-4677) ([845449f](845449f))
* normalize deep-dive dependency wording ([37a2bd0](37a2bd0))
* remove deep internal service details from public AGENTS ([a60e77a](a60e77a))
* strengthen workflow orchestration in AGENTS.md (DTCRCMERC-4677) ([b2ddef2](b2ddef2))
* update PR template and agent docs (DTCRCMERC-4677) ([9689af5](9689af5))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants