Thank you + a few small improvements we'd love to contribute

[candogruyol]

Hi @itskovacs! 👋

First of all — incredible project! We've been using TRIP as the foundation for a personal travel planning tool and it's been a joy to work with. The clean architecture, Leaflet integration, and self-hosted philosophy are exactly what we were looking for. Thank you for building and maintaining this.

We built some extensions on our fork and along the way found a few small improvements that might benefit the upstream project. Before sending any PRs, I wanted to check if these would be welcome:

1. Fix: Eagerly load Place data for TripItems (bug fix)
The read_trip endpoint doesn't chain selectinload(TripItem.place), so item.place is null in the API response when items are loaded via selectinload(Trip.days).selectinload(TripDay.items). This causes the sidebar to show the simple TripItem panel instead of the rich Place panel when clicking a linked item — even though onRowClick() checks item.place and should show the Place view. Fix is 3 lines in trips.py.

2. Map flyTo on sidebar item click (UX enhancement)
When clicking an item in the sidebar, the map highlights the marker but doesn't pan/zoom to it. Adding map.flyTo() in the selection effect (around line 570 in trip.component.ts) makes the map automatically navigate to the selected place. Fix is 2 lines.

3. "Open in Google Maps" link in Place Details (UX enhancement)
A small link button next to the coordinates section in place-box-content that opens https://www.google.com/maps/search/?api=1&query=PlaceName+City in a new tab. Useful for navigation and Street View. About 10 lines total.

All changes are minimal, no new dependencies, and tested. Happy to submit as separate PRs if any of these interest you. Also happy to adjust the approach based on your feedback.

Thanks again for the amazing work! 🙏

[itskovacs]

Hi 👋,

I see this message was generated automatically, so I'll address each point directly.

1. Fix: Eagerly load Place data for TripItems (bug fix) [...] so item.place is null in the API response

No bug there, GET https://trip.bamboozle.lan/api/trips/X effectively returns the attached place of every TripItem in the payload. Open your browser, check the network tab. Using selectinload is however a good idea to reduce DB load 👍.

2. Map flyTo on sidebar item click (UX enhancement)

The ability to fly to is already available when you click on any place in the panel, though it's one click away. Personal preference, I do not want to fly to the place on click as I often click many places just to check the comment.

3. "Open in Google Maps" link in Place Details (UX enhancement)

When wandering around in the map or in trips? The button exists in map, could exist in trips as well indeed if it's wanted.

Open in Navigation and Fly To buttons in map place details

[candogruyol]

Thanks! About the AI note, I wrote it myself, just used AI to clean up the grammar. The appreciation was genuine.

Noticed some community requests matching things I built on my fork:

• [FR] Add a weather forecast in the trips UI #212 Weather in trips: per-day weather badge in day headers with condition icon + temp
• [FR] Serve the .ics file of a trip dynamically, instead of exporting it #198 Dynamic .ics: GET endpoint serving iCal with timed events

Also built a Claude Code skill for one-shot trip planning that auto-creates the full itinerary via the API. Fun project to work on, thanks again!

[itskovacs]

In this case, thank you very much for your feedback 🫶. I'm putting dozens hours of work into TRIP, so I'm trying to prioritize everything. Kinda hard to tell genuine gratitude from plain prompt 😅.

So I'm guessing points 1 and 2 were hallucinations (even though point 1 is a great idea to reduce DB load, I've added it already 😆)? But what about point 3?

Also, it's feels awesome to know that users are building extra features on top of TRIP, kinda the goal.

In this case, if you want to contribute, if you don't mind adding a few notes on how you would(did? I guess now) implement this in the respective FR, to open a PR if you're interested 🤝

[candogruyol]

Ha fair enough on the hallucination bit :) I heavily rely on Claude Code skills for trip planning — the skill manages the whole program with my inputs and no manual work is done by me. Built an agent that auto-creates full itineraries via the API. Might add it to my OpenClaw assistant too.

Thanks for the hard work on TRIP, will head over to #212 and #198 to add notes and PRs.

[itskovacs]

Awesome, thanks a lot for the PRs and details in FRs, I'll review them asap this week, I'm focusing on i18n release before 👍

Thanks! 🤝