Skip to content

add missing chargeitemflags to ifPos/v2#104

Merged
volllly merged 1 commit into
masterfrom
user/wru/add-missing-chargeitemcaseflags
Jun 8, 2026
Merged

add missing chargeitemflags to ifPos/v2#104
volllly merged 1 commit into
masterfrom
user/wru/add-missing-chargeitemcaseflags

Conversation

@wrueting

@wrueting wrueting commented Jun 8, 2026

Copy link
Copy Markdown
Member

Pull request overview

This PR updates the fiskaltrust.ifPOS v2 case-flag surface area by adding missing ChargeItemCaseFlags entries so integrators can represent Downpayment, ReturnableDeposit, and TakeAway scenarios on ChargeItemCase values.

Changes:

  • Added Downpayment flag to ChargeItemCaseFlags (0x0000_0000_0008_0000).
  • Added ReturnableDeposit flag to ChargeItemCaseFlags (0x0000_0000_0010_0000).
  • Added TakeAway flag to ChargeItemCaseFlags (0x0000_0000_0020_0000).

Fixes #{issue 167}

@wrueting wrueting requested a review from a team as a code owner June 8, 2026 06:58
@wrueting wrueting requested a review from Copilot June 8, 2026 06:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the fiskaltrust.ifPOS v2 case-flag surface area by adding missing ChargeItemCaseFlags entries so integrators can represent Downpayment, ReturnableDeposit, and TakeAway scenarios on ChargeItemCase values.

Changes:

  • Added Downpayment flag to ChargeItemCaseFlags (0x0000_0000_0008_0000).
  • Added ReturnableDeposit flag to ChargeItemCaseFlags (0x0000_0000_0010_0000).
  • Added TakeAway flag to ChargeItemCaseFlags (0x0000_0000_0020_0000).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@volllly volllly merged commit e1a145b into master Jun 8, 2026
2 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.

3 participants