Skip to content

Fix default values - #132

Merged
reshmabidikar merged 1 commit into
masterfrom
work-for-ts-299
Oct 2, 2026
Merged

reshmabidikar merged 1 commit into
masterfrom
work-for-ts-299

Conversation

@reshmabidikar

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The change is straightforward, with only a minor brand-name spelling inconsistency noted.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Updates fallback invoice branding with readable company details instead of property-name placeholders.

Changes:

  • Adds default company name, address, location, country, and URL.
File Description
BrandingLoader.java Updates fallback company branding values.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

"companyCityProvincePostalCode", "companyCityProvincePostalCode",
"companyCountry", "companyCountry",
"companyUrl", "companyUrl",
"companyName", "Killbill, Inc.",
@reshmabidikar
reshmabidikar merged commit 7164aaf into master Oct 2, 2026
15 checks passed
@reshmabidikar
reshmabidikar deleted the work-for-ts-299 branch October 2, 2026 04:15
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