Skip to content

Docs updates for E/A#89379 Update Reveal button label on card details#91127

Merged
muttmuure merged 1 commit into
mainfrom
claude-docsUpdatesCardUIRevealButton
Jun 5, 2026
Merged

Docs updates for E/A#89379 Update Reveal button label on card details#91127
muttmuure merged 1 commit into
mainfrom
claude-docsUpdatesCardUIRevealButton

Conversation

@MelvinBot

Copy link
Copy Markdown
Contributor

Summary

  • Updates the help site article Cardholder-Settings-and-Features.md to change Reveal DetailsReveal, matching the button label change introduced in Update Expensify card UI #89379.

Test plan

  • Verify the help article renders correctly on HelpDot
  • Confirm the button label in the app matches the updated article text

The card details page button was renamed from "Reveal details" to
"Reveal" in #89379. Update the help article to match.

Co-authored-by: Shridhar Goel <ShridharGoel@users.noreply.github.com>
@MelvinBot MelvinBot added the HelpDot Apply this label if the issue relates to ExpensifyHelp label May 19, 2026
@MelvinBot MelvinBot mentioned this pull request May 19, 2026
52 tasks
@stephanieelliott stephanieelliott self-assigned this May 22, 2026
@stephanieelliott stephanieelliott marked this pull request as ready for review May 27, 2026 04:43
@github-actions

Copy link
Copy Markdown
Contributor

HelpDot Documentation Review

Overall Assessment

This PR makes a single, focused change: updating the button label from Reveal Details to Reveal in the Cardholder Settings and Features article, aligning the documentation with the UI change introduced in #89379. This is a straightforward terminology correction that directly supports the core governance principle of using exact UI text.

Scores Summary

  • Readability: 10/10 - The changed line remains clear, action-oriented, and well-structured within the existing numbered step sequence. No readability issues introduced.
  • AI Readiness: 10/10 - No impact on headings, metadata, or heading hierarchy. The change is scoped to step content only.
  • Style Compliance: 10/10 - The change correctly applies the Core UI Referencing Rules from HELPSITE_NAMING_CONVENTIONS.md: the button name is bolded, uses exact UI text, is not shortened or paraphrased, and is not wrapped in quotation marks.

Key Findings

  • The change follows the governance mandate to "use the exact text shown in the product UI" and to "match capitalization exactly." Updating Reveal Details to Reveal is the right fix given the upstream UI change in Update Expensify card UI #89379.
  • The surrounding step format (numbered, sequential, action-oriented) remains intact and compliant with HELP_AUTHORING_GUIDELINES.md Section 5.
  • No new governance issues are introduced by this change.

Recommendations

  • None for this PR. The change is minimal, correct, and well-scoped.

Files Reviewed

  • docs/articles/new-expensify/expensify-card/Cardholder-Settings-and-Features.md — Single line updated to match current UI button label. No issues found.

Note: This PR contained only one line change, so no inline comments were necessary.

@github-actions

Copy link
Copy Markdown
Contributor

HelpDot Documentation Review — Summary

PR: #91127
File changed: docs/articles/new-expensify/expensify-card/Cardholder-Settings-and-Features.md


Change Overview

Single-line update: **Reveal Details****Reveal** (line 82), updating the button label to match the current product UI.


Governance Compliance Scoring

Category Score Notes
UI Referencing (Naming Conventions) ✅ Pass Correctly updates to exact UI label per naming conventions
Button Formatting ✅ Pass Bold formatting preserved, sentence case maintained
Step Formatting ✅ Pass Sequential numbered steps remain intact
Heading Rules N/A No headings changed
Metadata N/A No metadata changed
AI Retrieval Optimization N/A No structural changes

Overall Score: ✅ Pass


Assessment

This is a clean, targeted fix that aligns the documentation with the current UI. The change follows the core rule from HELPSITE_NAMING_CONVENTIONS.md: "Use the exact text shown in the product UI." No other governance rules are impacted.

Recommendation: Approve as-is. No issues found with this change.


🤖 Generated with Claude Code

@stephanieelliott stephanieelliott requested a review from a team May 28, 2026 02:51
@melvin-bot

melvin-bot Bot commented May 28, 2026

Copy link
Copy Markdown

Concierge reviewer checklist:

  • I have verified the accuracy of the article
    • The article is within a hub that makes sense, and the navigation is correct
    • All processes, screenshots, and examples align with current product behavior.
    • All links within the doc have been verified for correct destination and functionality.
  • I have verified the readability of the article
    • The article's language is clear, concise, and free of jargon.
    • The grammar, spelling, and punctuation are correct.
    • The article contains at least one image, or that an image is not necessary
  • I have verified the formatting of the article
    • The article has consistent formatting (headings, bullet points, etc.) with other HelpDot articles and that aligns with the HelpDot formatting standards.
    • The article has proper text spacing and breaks for readability.
  • I have verified the article has the appropriate tone and style
    • The article's tone is professional, friendly, and suitable for the target audience.
    • The article's tone, terminology, and voice are consistent throughout.
  • I have verified the overall quality of the article
    • The article is not missing any information, nor does it contain redundant information.
    • The article fully addresses user needs.
  • I have verified that all requested improvements have been addressed

For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member?

cc @muttmuure

@melvin-bot melvin-bot Bot requested a review from muttmuure May 28, 2026 02:51
@melvin-bot

melvin-bot Bot commented May 28, 2026

Copy link
Copy Markdown

@muttmuure Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@melvin-bot melvin-bot Bot removed the request for review from a team May 28, 2026 02:51
@muttmuure muttmuure merged commit 647e140 into main Jun 5, 2026
21 of 23 checks passed
@muttmuure muttmuure deleted the claude-docsUpdatesCardUIRevealButton branch June 5, 2026 13:02
@melvin-bot melvin-bot Bot added the Emergency label Jun 5, 2026
@melvin-bot

melvin-bot Bot commented Jun 5, 2026

Copy link
Copy Markdown

@muttmuure looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

@OSBotify

OSBotify commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by https://github.com/muttmuure in version: 9.4.0-0 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Bundle Size Analysis (Sentry):

@OSBotify

OSBotify commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/Julesssss in version: 9.4.0-7 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Emergency HelpDot Apply this label if the issue relates to ExpensifyHelp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants