[No QA] [Payment due @ZhenjaHorbach] Update Discover section help docs to explain visibility behavior#87251
[No QA] [Payment due @ZhenjaHorbach] Update Discover section help docs to explain visibility behavior#87251MelvinBot wants to merge 1 commit into
Conversation
Add documentation that the Discover section disappears after the user watches the demo, and add a corresponding FAQ entry for consistency with other conditional Home sections. Co-authored-by: Yauheni Horbach <ZhenjaHorbach@users.noreply.github.com>
|
🎯 @ZhenjaHorbach, thanks for reviewing and testing this PR! 🎉 A payment issue will be created for your review once this PR is deployed to production. If payment is not needed (e.g., regression PR review fix etc), react with 👎 to this comment to prevent the payment issue from being created. |
|
|
||
| This section includes a short demo that introduces key areas of the app and shows how different parts fit together. This can be helpful when you’re getting started or returning after some time away. | ||
|
|
||
| Once you watch the demo, the Discover section is automatically removed from Home. |
There was a problem hiding this comment.
Naming and style: "Discover" refers to a UI section and should be bolded as Discover to match the formatting used throughout the rest of this article (e.g., lines 84, 37, 136). Per HELPSITE_NAMING_CONVENTIONS.md, section names must use exact UI text with bold formatting.
| Once you watch the demo, the Discover section is automatically removed from Home. | |
| Once you watch the demo, the **Discover** section is automatically removed from Home. |
HelpDot Documentation ReviewOverall AssessmentThis PR adds two small pieces of content to the Expensify Home Overview article: (1) a clarifying sentence in the Discover section explaining that it disappears after watching the demo, and (2) a new FAQ entry addressing why the Discover section might not be visible. Both additions are well-written, concise, and consistent with the existing article's tone and structure. Scores Summary
Key Findings
Recommendations
Files Reviewed
|
|
A preview of your ExpensifyHelp changes have been deployed to https://72af698c.helpdot.pages.dev ⚡️ Updated articles: |
|
Looking at this change I think it is not necessary to document |
Explanation of Change
Updates the Expensify Home Overview help article to document that the Discover section on Home disappears after the user watches the demo. Also adds a FAQ entry ("Why don't I see the Discover section?") for consistency with other conditional Home sections (Time-sensitive, Spend over time, Assigned cards) that already have such FAQ entries.
Fixed Issues
$ #87224
Tests
docs/articles/new-expensify/getting-started/Expensify-Home-Overview.mdOffline tests
N/A (docs-only change)
QA Steps
N/A (docs-only change)
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectiontoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
N/A (docs-only change)
Co-authored-by: ZhenjaHorbach 68128028+ZhenjaHorbach@users.noreply.github.com