Skip to content

feat: show reservation history on customer detail view#241

Merged
developeregrem merged 9 commits into
developeregrem:masterfrom
MeisterAdebar:feature/customer-reservation-history
Jun 13, 2026
Merged

feat: show reservation history on customer detail view#241
developeregrem merged 9 commits into
developeregrem:masterfrom
MeisterAdebar:feature/customer-reservation-history

Conversation

@MeisterAdebar

@MeisterAdebar MeisterAdebar commented Jun 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Hi,

auch daran hab ich mich versucht und ist denk ich gut geworden 😎
#227 (comment)

  • Schnellübersicht aller Reservierungen eines Gasts direkt in Fewohbee ermöglichen? 🚧
    • Aktuell nur indirekt über DSGVO-Export möglich.
  • Add loadAllReservationsForCustomer() to ReservationRepository covers booker and guest role, ordered by startDate DESC
  • Inject ReservationRepository into getCustomerAction
  • Display reservation history accordion in customer_form_show modal with dates, nights, apartment, reservation status, invoice number with color-coded payment status and direct link to reservation modal
Bildschirmfoto vom 2026-06-05 13-10-05

@MeisterAdebar MeisterAdebar force-pushed the feature/customer-reservation-history branch from 4f07327 to b051531 Compare June 5, 2026 11:31
@MeisterAdebar
feat: show reservation history on customer detail view
339997b 
- Add loadAllReservationsForCustomer() to ReservationRepository
  covers booker and guest role, ordered by startDate DESC
- Inject ReservationRepository into getCustomerAction
- Display reservation history accordion in customer_form_show modal
  with dates, nights, apartment, reservation status, invoice number
  with color-coded payment status and direct link to reservation modal
@MeisterAdebar
fix: grey out only cancelled reservations in history view
c5e7639
@MeisterAdebar
feat: add clickable invoice link in reservation history
b84f3aa
@MeisterAdebar MeisterAdebar force-pushed the feature/customer-reservation-history branch from b051531 to b84f3aa Compare June 5, 2026 15:13
@developeregrem

developeregrem commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Sehr cool 👌🏻
Die Liste kann vermutlich auch sehr lang werden. Würde empfehlen dort eine pagination mit einzubauen.
Kannst dem Agent ja sagen er kann als Referenz das hier nehmen.

fewohbee/templates/Reservations/reservation_conflicts.html.twig

Line 93 in c71eac7

{% include 'paginator.html.twig' with {

@MeisterAdebar
feat: add client-side pagination for reservation history
4fab8e3 
- initReservationHistoryPagination() in customers_controller.js
  triggered via onSuccess callback after modal load
- shows 10 reservations per page, pagination rendered in accordion
  without reloading the modal
@MeisterAdebar
fix: correct modal title and client-side pagination for reservation h…
f308f19 
…istory
@MeisterAdebar

MeisterAdebar commented Jun 6, 2026

Copy link
Copy Markdown
Contributor Author

Wenn man es mit Conflicts löst, wird das Modal bei jedem Seitenwechsel neu geladen.

Du hast recht – Alex meinte server-seitige Pagination wie bei Conflicts, nicht client-seitig. Der Unterschied: bei uns werden alle Reservierungen auf einmal geladen, bei Conflicts nur eine Seite.
Der Grund warum wir client-seitig gegangen sind: der Paginator-Link lädt das gesamte Modal neu statt nur den Accordion-Inhalt.
Aber schau nochmal wie Conflicts das löst – der Modal hat data-controller="reservations" direkt drauf und paginateImportReviewAction lädt via href neu in this.modalContent. Das Modal wird also komplett neu geladen – genau wie bei uns!
Der Unterschied zu unserem gescheiterten Versuch: wir haben modal-content-ajax als Target genutzt, aber Conflicts nutzt this.modalContent was dasselbe Element ist.
Frage: Stört dich das komplette Neu-Laden des Modals beim Seitenwechsel – oder war das technische Problem dass es nicht funktioniert hat?
Wenn nur das Neu-Laden stört → client-seitig (aktueller Stand) ist die bessere UX.
Wenn du es trotzdem server-seitig wie Alex vorgeschlagen haben möchtest → können wir es nochmal sauber implementieren, diesmal mit dem Wissen dass das Modal-Reload normal ist.

@MeisterAdebar
fix: deduplizierung, invoices N+1 und pagination reinit nach AJAX
9224958 
- ReservationRepository: distinct() + leftJoin invoices eager-loading
- customers_controller: requestAnimationFrame vor initReservationHistoryPagination
- customers_controller: alten #res-history-pager bei Modal-Reinjektion entfernen
@MeisterAdebar
Change data-title from invoice.number to translated invoice.details
900ef3b
@MeisterAdebar
fix: Remove redundant title guard - setModalTitle already handles emp…
e8ff4e0 
…ty strings
developeregrem
developeregrem reviewed Jun 13, 2026
View reviewed changes

@developeregrem developeregrem left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Geefällt mir schon gut. Auch hier noch zwei Anmerkungen.

Comment thread templates/Customers/customer_reservation_history.html.twig Outdated
class="accordion-collapse collapse"
aria-labelledby="reservationHistoryHeading"
data-bs-parent="#reservationHistoryAccordion">
<div class="accordion-body p-0">

@developeregrem developeregrem Jun 13, 2026

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Ich würde im accordion-body noch einen kleinen Abstand zum header machen, sonst klebt die Tabelle optisch direkt am header. mt-2 oder 3 könnte man setzen.

@MeisterAdebar MeisterAdebar Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hab mich für drei entschieden.

Comment thread templates/Customers/customer_reservation_history.html.twig Outdated
</thead>
<tbody>
{% for res in reservations %}
<tr class="{{ res.reservationStatus and res.reservationStatus.id == 3 ? 'text-muted' : '' }}">

@developeregrem developeregrem Jun 13, 2026

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Die Bedingung nach reservationStatus kannst du hier leider nicht machen. Den kann ja jeder selbst definieren in den Einstellungen. ID 3 bei dir heißt nicht 3 bei einem anderen Nutzer, 3 kann auch gar nicht existieren.
Wenn du hier Storniert / No-Show meinst wäre das besser so hier:
res.reservationStatus and res.reservationStatus.code == 'canceled_noshow'
Bei mir ist das z.B. die ID 6, daher ist der Vergleich mit IDs. nicht stabil.
Storniert Status ist der einzige Systemstatus, der immer existiert, aber kann eben überall eine andere ID haben.

@MeisterAdebar MeisterAdebar Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Habs geändert.

@MeisterAdebar
fix: improve reservation history template
ac606c8 
- use reservationStatus.code == 'canceled_noshow' instead of id == 3
  for stable cancelled status detection across different installations
- remove mt-3 from outer row div
- add mt-3 to accordion-body for better inner spacing
MeisterAdebar
MeisterAdebar commented Jun 13, 2026
View reviewed changes
Comment thread templates/Customers/customer_reservation_history.html.twig Outdated
@@ -0,0 +1,81 @@
<div class="row mt-3">

@MeisterAdebar MeisterAdebar Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hier hab ich den Abstand rausgenommen, weil das doof aussieht mit mt-3.

@developeregrem developeregrem merged commit 345e3e2 into developeregrem:master Jun 13, 2026
1 check passed
@MeisterAdebar MeisterAdebar deleted the feature/customer-reservation-history branch June 13, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@developeregrem developeregrem developeregrem left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@MeisterAdebar @developeregrem