Skip to content

no-issue: add adjustment return tax and response field parsing - #13

Merged
reshmabidikar merged 1 commit into
killbill:mainfrom
JeffTax:chore/no-ref/sync-adj-external-id
Sep 21, 2026
Merged

reshmabidikar merged 1 commit into
killbill:mainfrom
JeffTax:chore/no-ref/sync-adj-external-id

Conversation

@JeffTax

@JeffTax JeffTax commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Ports recent invoice-tax fixes onto upstream main
  • Adds return estimates for untaxed ITEM_ADJ / REPAIR_ADJ (negative TAX linked to the adj item; {invoiceId}:adj-return document id)
  • Uses shared EXTERNAL_CHARGE product external id for plan-less external charges
  • Accepts line_external_id / rate_percentage (with fallback to external_id / rate) on tax line responses
  • Skips empty sales POSTs on adj/credit-only invoices; updates Aviate follow-up notes for the shipped return path
  • Lands green Dependabot bumps: Jackson 2.22.x (annotations/core/databind aligned), wiremock 3.0.1, surefire 3.6.0, checkout v7, setup-java v6, codeql-action v4

Tests

  • JAVA_HOME=…/openjdk@11 mvn -B -ntp test — 62 tests, 0 failures
  • New/updated unit coverage in TestInvoiceRequestMapper and TestInvoiceTaxIdempotency (return estimate, external charge identity, adj-only skip)

@JeffTax

JeffTax commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

Pushed an additional commit onto this branch with the Dependabot dependency fixes (Jackson annotations/core/databind aligned to 2.22.x, wiremock 3.0.1, surefire 3.6.0, GHA checkout/setup-java/codeql bumps).

That supersedes #14 and the open Dependabot PRs #3#6 / #8#10. #7 (oss-parent 0.147) stays out — Jakarta/JDK 21 migration, not a drop-in bump.

@JeffTax JeffTax changed the title no-issue: add adjustment return tax and mosaic response fixes no-issue: add adjustment return tax and response fixes Sep 18, 2026
@JeffTax
JeffTax force-pushed the chore/no-ref/sync-adj-external-id branch from e3f5d69 to 5e0fec8 Compare September 18, 2026 15:31
@JeffTax JeffTax changed the title no-issue: add adjustment return tax and response fixes no-issue: add adjustment return tax and response field parsing Sep 18, 2026
Port invoice-tax fixes: ITEM_ADJ/REPAIR_ADJ return estimates, EXTERNAL_CHARGE
product identity, alternate tax-line field names, and skip empty sales POSTs.
Bump Jackson annotations/core/databind together and apply the other green
Dependabot upgrades so CI stays green. Leave oss-parent at 0.146.x.
@JeffTax
JeffTax force-pushed the chore/no-ref/sync-adj-external-id branch from 667edbb to d5e6d35 Compare September 18, 2026 20:34
@reshmabidikar
reshmabidikar merged commit d0445a8 into killbill:main Sep 21, 2026
6 checks passed
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.

2 participants