Vereins-IT/ezgg-lan-manager
4
1
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

bugfix/rounding-taxes #10

Merged
tcprod merged 1 commits from bugfix/rounding-taxes into main 2025-02-07 22:31:50 +00:00
Conversation 3 Commits 1 Files Changed 24 +342 -235
tcprod commented 2025-02-04 18:07:48 +00:00
Owner
Copy Link
No description provided.
tcprod added 3 commits 2025-02-04 18:07:48 +00:00
Typhus reviewed 2025-02-04 19:12:45 +00:00
Typhus left a comment
Owner
Copy Link

Example config hat noch falsche Beispielwerte für den Ticket-Preis

Example config hat noch falsche Beispielwerte für den Ticket-Preis
👍 1
Typhus requested changes 2025-02-04 19:33:25 +00:00
Dismissed
src/ez_lan_manager/helpers/create_demo_database_content.py Outdated
@ -17,0 +16,4 @@
{"user_name": "jason", "user_mail": "juergen@demomail.com", "password_clear_text": "jason"},
# Gast + Ticket(NORMAL) + Sitzplatz
{"user_name": "lisa", "user_mail": "lisa@demomail.com", "password_clear_text": "lisa"}, # Teamler
{"user_name": "thomas", "user_mail": "thomas@demomail.com", "password_clear_text": "thomas"} # Teamler + Admin
Typhus commented 2025-02-04 19:23:42 +00:00
Owner
Copy Link

Diese Umbrüche gefallen mir nicht

Diese Umbrüche gefallen mir nicht
tcprod marked this conversation as resolved
src/ez_lan_manager/services/AccountingService.py Outdated
@ -82,2 +68,2 @@
return result
def make_euro_string_from_decimal(euros: Decimal) -> str:
""" Internally, all money values are cents as ints. Only when showing them to the user we generate a string.
Typhus commented 2025-02-04 19:26:02 +00:00
Owner
Copy Link

Mehrzeilige Docstrings bitte bitte so formatieren:

    def abc():
    """
    Hallo, ich bin ein Docstring,
    der aus mehrere Zeilen gebrochen ist.
    """
    pass
Mehrzeilige Docstrings bitte bitte so formatieren: ```py def abc(): """ Hallo, ich bin ein Docstring, der aus mehrere Zeilen gebrochen ist. """ pass ```
tcprod marked this conversation as resolved
Typhus commented 2025-02-04 19:34:37 +00:00
Owner
Copy Link

Auf der Arbeit hätte ich den MR so abgelehnt. Bitte ändere keine Formatierung für Codezeilen deren Logik du nicht angefasst hast. Der MR war schon großm genug, auch ohne das ich versuche den Unterschied zu finden den es nicht gibt.

Auf der Arbeit hätte ich den MR so abgelehnt. Bitte ändere keine Formatierung für Codezeilen deren Logik du nicht angefasst hast. Der MR war schon großm genug, auch ohne das ich versuche den Unterschied zu finden den es nicht gibt.
👍 1
tcprod commented 2025-02-05 07:58:07 +00:00
Author
Owner
Copy Link

Auf der Arbeit hätte ich den MR so abgelehnt. Bitte ändere keine Formatierung für Codezeilen deren Logik du nicht angefasst hast. Der MR war schon großm genug, auch ohne das ich versuche den Unterschied zu finden den es nicht gibt.

Sorry, allerdings war mein PyCharm total bunt, vielleicht können wir uns irgendwie die Config von PyCharm gleich einstellen.

> Auf der Arbeit hätte ich den MR so abgelehnt. Bitte ändere keine Formatierung für Codezeilen deren Logik du nicht angefasst hast. Der MR war schon großm genug, auch ohne das ich versuche den Unterschied zu finden den es nicht gibt. Sorry, allerdings war mein PyCharm total bunt, vielleicht können wir uns irgendwie die Config von PyCharm gleich einstellen.
tcprod added 1 commit 2025-02-06 14:41:37 +00:00
tcprod commented 2025-02-06 14:42:08 +00:00
Author
Owner
Copy Link

Habe nun alle review Punkte bearbeitet.

Habe nun alle review Punkte bearbeitet.
tcprod closed this pull request 2025-02-06 14:42:08 +00:00
tcprod reopened this pull request 2025-02-06 14:42:13 +00:00
tcprod added 1 commit 2025-02-06 14:44:34 +00:00
tcprod started working 2025-02-06 14:45:05 +00:00
tcprod stopped working 2025-02-06 14:45:14 +00:00
9 seconds
Typhus reviewed 2025-02-06 15:13:59 +00:00
src/ez_lan_manager/services/AccountingService.py Outdated
@ -83,1 +68,3 @@
return result
def make_euro_string_from_decimal(euros: Decimal) -> str:
"""
Internally, all money values are cents as ints. Only when showing them to the user we generate a string.
Typhus commented 2025-02-06 15:13:59 +00:00
Owner
Copy Link

Ich hatte gehofft das du beim ändern des Docstrings auch den Inhalt anpasst. Denn der ist jetzt nicht mehr korrekt.

Ich hatte gehofft das du beim ändern des Docstrings auch den Inhalt anpasst. Denn der ist jetzt nicht mehr korrekt.
tcprod commented 2025-02-07 17:58:36 +00:00
Author
Owner
Copy Link

done.

done.
Typhus marked this conversation as resolved
tcprod added 1 commit 2025-02-07 17:58:36 +00:00
Typhus requested review from Typhus 2025-02-07 18:33:16 +00:00
Typhus approved these changes 2025-02-07 18:33:28 +00:00
tcprod force-pushed bugfix/rounding-taxes from afe5575b34 to a419ee8885 2025-02-07 22:29:11 +00:00 Compare
tcprod merged commit 40f8bc1049 into main 2025-02-07 22:31:50 +00:00
tcprod deleted branch bugfix/rounding-taxes 2025-02-07 22:31:50 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Typhus
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Total Time Spent: 9 seconds
tcprod
9 seconds
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Vereins-IT/ezgg-lan-manager#10
No description provided.
Delete Branch "bugfix/rounding-taxes"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?