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.

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

Fix #662 und #663 Debitorenbuchung Design40 Fehlbetrag setzen und Zah… #298

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arevamp
Copy link
Contributor

@arevamp arevamp commented Mar 7, 2024

…lung buchen

@arevamp arevamp requested a review from bblessmann March 7, 2024 16:41
Copy link
Member

@bblessmann bblessmann left a comment

Choose a reason for hiding this comment

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

Hallo Andeas,

Hast Du das getetstet?

Das sieht irgendwie kaputt aus.
Einige Anmerkungen:

  • eine nun gebuchte Zahlung wird nicht angezeigt. Das klappte vor Deinem commit noch
  • Du setzt die Namen und ids auf irgendwas mit temp. Und temp verwendet row aus der loop, anstatt loop.count. row ist aber ein Hash aus payments und keine Zahl oder String

Und bitte die Tickets nicht schließen, bevor das im Master ist.
Zudem wäre eine commit-Message, die den Fix beschreibt gut und nicht nur der Hinweis auf den Bugtracker und Fehler.
Zusätzlich kannst Du ja sowas wie "behebt #662" in den commit schreiben.

Viele Grüße
Bernd

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