Skip to content

Apeinvoice - #13

Open
cngan-wd wants to merge 4 commits into
Workday:mainfrom
cngan-wd:apeinvoice
Open

Apeinvoice#13
cngan-wd wants to merge 4 commits into
Workday:mainfrom
cngan-wd:apeinvoice

Conversation

@cngan-wd

Copy link
Copy Markdown

What this PR adds or changes

Adding AP E Invoice Reference Application

Checklist for new or changed examples

  • The example lives entirely in its own folder under examples/
  • node scripts/validate-examples.mjs --check passes (valid example.json, README present, index table in sync)
  • The README says what the artifact is, how to use it, and what to change before deploying it (the "Before you deploy" section)
  • No credentials, tenant names, or real personal data anywhere in the folder
  • node scripts/audit-examples.mjs --changed passes, or the findings it reports are explained below (see docs/EXAMPLE_BEST_PRACTICES.md)

Anything reviewers should know?

None

@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown

The Audit examples check failed because of the items under Fix before merge. Push a fix and it re-runs automatically.

Example audit

Audited catalog/ap-einvoice. Passed: no hardcoded Workday URLs or app ids; no debug logging.

1 to fix and 0 suggestion(s).

Fix before merge

Where Rule What to change
catalog/ap-einvoice/example.json HubExampleJsonRule example.json is not valid JSON: Unexpected token 'C', ..."uthors": [Chris Ngan"... is not valid JSON

Run it yourself: ./scripts/install-arcane.sh once, then node scripts/audit-examples.mjs --changed. Rule explanations and fixes: docs/EXAMPLE_BEST_PRACTICES.md.

ACTION items fail the Audit examples check. ADVICE never blocks. Maintainers can add the audit-override label to merge with open ACTION items.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant