Skip to content

bugfix: render single day in ItineraryDay instead of mapping all itinerary days#41

Open
nleiby wants to merge 1 commit intorawcomposition:mainfrom
nleiby:pr/1-itinerary-day-fix
Open

bugfix: render single day in ItineraryDay instead of mapping all itinerary days#41
nleiby wants to merge 1 commit intorawcomposition:mainfrom
nleiby:pr/1-itinerary-day-fix

Conversation

@nleiby
Copy link

@nleiby nleiby commented Feb 1, 2026

Currently, adding a day to the itinerary adds all days for each day in the itinerary (screen shot and screen capture attached)

This PR:

  1. Adds 3 lines before the return: dayIndex, date, and const { notes, locations } = day.
  2. Returns a single root
    (no <> or .map()).
  3. Uses dayIndex + 1 for the heading, day.id in TravelTime and in open("addItineraryLocation", { dayId: day.id }).

Current behavior:
image

Screen.Recording.2026-02-01.at.4.05.49.PM.mov

…ry days

Co-authored-by: Cursor <cursoragent@cursor.com>
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