Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Clarify Amount calculation #323

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Clarify Amount calculation #323

wants to merge 4 commits into from

Conversation

forsthug
Copy link
Contributor

@forsthug forsthug commented Apr 26, 2024 •
edited

Clarify Amount calculation

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

{Summary of the changes}

Description

{Detail}

Fixes #{issue number}

@forsthug
Update data-structures.md
e0b2324 
Clarify Amount calculation
@forsthug forsthug requested a review from a team as a code owner April 26, 2024 10:23
@forsthug @TSchmiedlechner
Update doc/middleware-de-kassensichv/data-structures/data-structures.md
2682b87 
Co-authored-by: Tom Schmiedlechner <TSchmiedlechner@users.noreply.github.com>
@forsthug @TSchmiedlechner
Update doc/middleware-de-kassensichv/data-structures/data-structures.md
9db7050 
Co-authored-by: Tom Schmiedlechner <TSchmiedlechner@users.noreply.github.com>
@forsthug forsthug enabled auto-merge April 29, 2024 08:05
@volllly
Copy link
Contributor

volllly commented May 6, 2024

/azp run

@TSchmiedlechner TSchmiedlechner marked this pull request as draft May 6, 2024 09:47
auto-merge was automatically disabled May 6, 2024 09:47

Pull request was converted to draft

@TSchmiedlechner TSchmiedlechner marked this pull request as ready for review May 6, 2024 09:47
@TSchmiedlechner
Copy link
Member

/azp run

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

@TSchmiedlechner TSchmiedlechner TSchmiedlechner approved these changes

@Bluette-C-Riviere Bluette-C-Riviere Bluette-C-Riviere approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@forsthug @volllly @TSchmiedlechner @Bluette-C-Riviere