#10 added docs about how payment vendor side tip is handled + extend …#11
Merged
deboragracio merged 17 commits intomainfrom Mar 2, 2026
Merged
#10 added docs about how payment vendor side tip is handled + extend …#11deboragracio merged 17 commits intomainfrom
deboragracio merged 17 commits intomainfrom
Conversation
…HOWTO_01 and HOWTO_08 to log the tip information nicely on the console
There was a problem hiding this comment.
Pull request overview
This pull request enhances payment handling logging and documentation, specifically focusing on scenarios where payment vendors add tips. The changes improve traceability by logging both requested and received payment amounts and provide comprehensive documentation explaining how tip scenarios work.
Changes:
- Enhanced
Utils.DumpToLoggermethod to accept optionalPayItemRequestparameter and display tip calculations - Added detailed documentation in
HOWTO_01_Payment_csharp/README.MDexplaining payment vendor tip handling with a flow example - Refactored payment examples to declare
PayItemRequestobjects before async operations for better logging context
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| libPosSystemAPI/PosAPIUtils/Utils.cs | Enhanced DumpToLogger method to accept optional PayItemRequest parameter and log requested vs received amounts with tip calculations |
| libPosSystemAPI.Test/IntegrationTestsPayment.cs | Simplified test comment to remove tip scenario reference |
| README.MD | Updated test case documentation to reference new tip handling guide |
| HOWTO_01_Payment_csharp/README.MD | Added comprehensive section explaining payment vendor tip handling with flow example |
| HOWTO_08_pay_sign_issue_csharp/Program.cs | Refactored to declare PayItemRequest before async operation and pass to DumpToLogger |
| HOWTO_01_Payment_csharp/Program.cs | Refactored to declare PayItemRequest before async operation and pass to DumpToLogger |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
deboragracio
left a comment
There was a problem hiding this comment.
Hi Thomas, here's my review for the text.
HOWTO_01_Payment_csharp/README.MD
Outdated
| - The guest sees a TIP entry screen and adds a tip of 2€ | ||
| - The guests now pays the full sum of 12€ | ||
| - The payment app reports a paid amount of 12€ (including 2€ of tip) | ||
| - The fiskaltrust InStore App / POS System API does report back the following in the payment response (see also example in the POS System API docs): |
Contributor
There was a problem hiding this comment.
Should a hyperlink be added to: (see also an example on the POS System API docs)?
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
Co-authored-by: deboragracio <63159404+deboragracio@users.noreply.github.com>
MaximilianFT
approved these changes
Mar 2, 2026
deboragracio
approved these changes
Mar 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces improvements to payment handling, especially around payments that include tips, and enhances logging for better traceability of payment requests and responses. It also updates documentation to explain how to handle tip scenarios and clarifies test and example code to reflect these changes.
Enhancements to payment handling and logging:
Utils.DumpToLoggermethod inUtils.csto accept the originalPayItemRequestand log both the requested and received amounts, including calculation and display of any included tip amount. The log output is now more structured and easier to read, especially for responses with multiple pay items. [1] [2] [3] [4]Utils.DumpToLoggerin payment examples (Program.csin bothHOWTO_01_Payment_csharpandHOWTO_08_pay_sign_issue_csharp) to pass the original payment request, ensuring that logs contain complete context for each payment. [1] [2]Documentation updates for tip handling:
HOWTO_01_Payment_csharp/README.MDexplaining how to handle payments where a tip is added by the payment vendor, including a flow example and explanation of how the tip is represented in the response.README.MDto reference the new documentation for handling payments with tips, specifically for the test case with code30000,50.Code and test clarifications: