Skip to content

Docs updates for E/A#92202 Document Sage Intacct travel invoicing reconciliation - #92802

Closed
MelvinBot wants to merge 1 commit into
mainfrom
docs-intacct-travel-invoicing-cr
Closed

MelvinBot wants to merge 1 commit into
mainfrom
docs-intacct-travel-invoicing-cr

Conversation

@MelvinBot

Copy link
Copy Markdown
Contributor

Summary

Adds help-site documentation for the Sync travel invoicing settlements advanced setting added to the Sage Intacct connection in #92202.

  • Documents the new Sync travel invoicing settlements toggle in the Sage Intacct Advanced settings section, including its dependency on Auto-sync.
  • Documents the Reconciliation account picker shown when the toggle is enabled.
  • Cross-links to the existing Travel Invoicing enablement article as the prerequisite workflow.

Changed files

  • docs/articles/new-expensify/connections/sage-intacct/Configure-Sage-Intacct.md

Authoring compliance

  • UI labels (Sync travel invoicing settlements, Reconciliation account, Auto-sync) match the App source strings exactly and use Sentence case per HELPSITE_NAMING_CONVENTIONS.md.
  • New ## subsection matches the existing sibling structure under the Advanced settings section.
  • Toggle/menu labels are bolded; cross-link uses a descriptive relative link per HELP_AUTHORING_GUIDELINES.md.

Note: This feature is gated behind Travel Invoicing being enabled and Expensify Cards being provisioned, so it could not be exercised in a live UI session. The labels were verified against the App source strings (src/languages/en.ts), which are what the live UI renders.

@allgandalf, please review the files changed and confirm they reflect the current behavior. Then mark this PR Ready for review.

Add a Sync travel invoicing settlements subsection to the Sage Intacct
Advanced settings docs, covering the new toggle and Reconciliation
account picker added in #92202.

Co-authored-by: Gandalf <allgandalf@users.noreply.github.com>
@MelvinBot MelvinBot added the HelpDot Apply this label if the issue relates to ExpensifyHelp label Jun 5, 2026
@MelvinBot

Copy link
Copy Markdown
Contributor Author

The failing build check is not caused by this PR. It's an environmental failure on the shared Deploy ExpensifyHelp workflow: the Install pcregrep step (sudo apt-get install -y pcregrep) is failing with E: Unable to locate package pcregrep on the Blacksmith ubuntu-2404 runner.

This PR only changes one Markdown file (docs/.../Configure-Sage-Intacct.md) and doesn't touch the workflow — so I have not pushed a change to this branch (bundling a shared CI fix into a docs PR would be out of scope, and it wouldn't fix the other affected PRs).

Evidence it's environmental, not this PR
  • The identical Install pcregrep step succeeded on main at 15:05 UTC today (run 27022788332) and then started failing on every docs PR at ~18:19–18:21 UTC. Same command, same runner OS, no change to the step.
  • The workflow file on this branch is byte-identical to origin/main (deployExpensifyHelp.yml:40-41, runner :33).
  • The same failure is hitting 5+ docs PRs simultaneously (e.g. docs-xero-travel-invoicing-cr, melvin-docs-auditor-readonly-workspace, claude-docs-qbo-travel-invoicing-cr), which is the signature of a shared-runner/infra issue rather than a per-PR bug.

pcregrep is used by enforceVideoFormats.sh:44. Because the package installed fine hours earlier on the same OS, it hasn't been removed from the archive — the runner's apt package index is just stale/empty, so apt-get install can't locate it.

Recommended fix (separate infra PR on main, not this PR): add sudo apt-get update before the install:

- name: Install pcregrep
  run: sudo apt-get update && sudo apt-get install -y pcregrep

The workflow currently never runs apt-get update, so it only works while the runner image happens to have a warm package index.

What you can do now:

  • Re-running the check may pass if it lands on a runner with a warm index, but it isn't a reliable fix.
  • @MelvinBot open an infra PR to fix this and I'll raise a dedicated PR to add apt-get update to deployExpensifyHelp.yml. Once that merges to main and you merge main into this branch, the check will pass.

@allgandalf allgandalf closed this Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

HelpDot Apply this label if the issue relates to ExpensifyHelp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants