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 #1229 (Vergangene Buchungen in Buchungsübersicht) #1554

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

Conversation

hansmorb
Copy link
Contributor

Was

Die Buchungsliste zeigt mit der Initialkonfiguration nur Buchungen ab dem heutigen Tag an. Wenn vergangene Buchungen angezeigt werden sollen, muss über den Filter das Startdatum in die Vergangenheit gesetzt werden. Eben dieser Filter wird aber nicht angezeigt, wenn keine Buchungen vorhanden sind. Deshalb sorgt dieser PR dafür, dass bei nicht gesetztem Startdatum und keinen Buchungen in der Zukunft sukzessive die letzten drei Jahre durchsucht werden um Nutzenden wenigstens eine Buchung anzeigen zu können. Von da aus können Nutzende dann mit dem ihnen nun angezeigten Filter auch weiter in der Vergangenheit ihre Buchungen sehen.

  • Mögliche Probleme *

Wenn Nutzende keine Buchungen in den letzten drei Jahren getätigt haben wird ihnen die Buchungsliste wieder nicht angezeigt.

  • Alternative *
    Eine schöne Alternative wäre bestimmt auch neben dem Warnhinweis "Keine Buchungen vorhanden" einen Link zu haben "Alte Buchungen laden", der dann die entsprechende AJAX Anfrage stellt. Das würde ein Verwirren der Nutzer durch alte Buchungen vielleicht verhindern. Das war jetzt eine Lösung mit relativ geringem Aufwand.

closes #1229

@hansmorb hansmorb added enhancement New feature or request php Pull requests that update Php code labels Mar 14, 2024
Copy link

codecov bot commented Mar 14, 2024

Codecov Report

Attention: Patch coverage is 12.50000% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 45.65%. Comparing base (72ead46) to head (71ebf85).
Report is 155 commits behind head on master.

Files Patch % Lines
src/View/Booking.php 12.50% 14 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1554      +/-   ##
============================================
- Coverage     45.70%   45.65%   -0.05%     
- Complexity     2560     2565       +5     
============================================
  Files            93       93              
  Lines         10095    10110      +15     
============================================
+ Hits           4614     4616       +2     
- Misses         5481     5494      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hansmorb hansmorb added this to the 2.9.3 milestone Apr 26, 2024
@hansmorb hansmorb added the question Further information is requested label May 22, 2024
@hansmorb hansmorb removed this from the 2.9.3 milestone May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request php Pull requests that update Php code question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vergangene Buchungen in Buchungsübersicht
1 participant