Repository navigation
Conversation
Co-authored-by: Mohammad Luthfi Fathur Rahman <mollfpr@users.noreply.github.com>
40 tasks done
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Mohammad Luthfi Fathur Rahman <mollfpr@users.noreply.github.com>
Contributor
|
A preview of your ExpensifyHelp changes have been deployed to https://3ea8ae28.helpdot.pages.dev ⚡️ Updated articles: |
3 tasks
Contributor
|
No update needed on this, per tightened Melvinbot criteria (see https://github.com/Expensify/Expensify/issues/685679). Please comment or reopen if you disagree! |
truph01
pushed a commit
to truph01/App
that referenced
this pull request
Sep 28, 2026
…nsify/workspaces
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Explanation of Change
Help site updates for the invoice field behavior changed in #100850.
That PR made two things visible to users that the help site does not document, and the help site had no article covering invoice fields at all:
The third fix in that PR (hiding the
Fixbadge on paid invoices in the LHN) restores intended behavior and needs no help site change.Changes in this PR:
docs/articles/new-expensify/workspaces/Add-Invoice-Fields.md— covers who can add invoice fields, enabling Invoices and Invoice fields, adding a field, the shared name rule between invoice fields and report fields, editing and deleting a field, and how invoice fields carry over when settings are copied.Enable-Report-Fields.md— adds a note that report field names must be unique across the workspace and share the same set of names as invoice fields.Copy-Workspace-Settings.md— adds an FAQ confirming invoice fields are copied when Invoices is selected, and that the row shows the count.Duplicate-Workspace.md— clarifies that invoice fields do carry over, alongside the existing FAQ stating invoicing details do not.On the error copy: deploy blocker #100876 reports that the duplicate-name error names the field type of the page you're on rather than the field that actually holds the name. These articles therefore describe the error generically ("an error telling you a field with that name already exists") and tell users to check both their invoice fields and their report fields. That stays accurate whichever way the copy lands, so the articles won't need a follow-up edit when that blocker is fixed.
All other UI labels were verified against a live Control workspace on web before writing: More features > Earn > Invoices; the workspace Invoices page and its Invoice fields toggle; the Add field button with Name and Type; and the
1 invoice fieldstext on the Select settings page. The articles followdocs/HELPSITE_NAMING_CONVENTIONS.md,docs/HELP_AUTHORING_GUIDELINES.md, anddocs/TEMPLATE.md.Fixed Issues
$ #100850
PROPOSAL:
Tests
// TODO: The human co-author must fill out the tests you ran before marking this PR as "ready for review"
// Please describe what tests you performed that validates your changed worked.
Documentation-only change — no product code is touched. Reviewing means confirming the articles match current behavior:
docs/articles/new-expensify/workspaces/Add-Invoice-Fields.mdagainst a Control workspace with Invoices enabled and confirm every label and navigation path matches the live UI.Enable-Report-Fields.mdmatches the behavior shipped in [CP Staging] Fix invoice field visibility, count, and name collision issues #100850.Copy-Workspace-Settings.mdandDuplicate-Workspace.mdmatches what the Invoices row shows.Offline tests
N/A — documentation only.
QA Steps
// TODO: The human co-author must fill out the QA tests you ran before marking this PR as "ready for review".
// Please describe what QA needs to do to validate your changes and what areas do they need to test for regressions.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
@mollfpr, please review the files changed and confirm they reflect the current behavior. Then post a new comment saying
@MelvinBot mark this PR as ready for review.