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

refactor(itinerary): adapt itinerary-page component to upstream #826

Merged
merged 4 commits into from
Dec 9, 2024

Conversation

andreashelms
Copy link
Collaborator

@andreashelms andreashelms commented Nov 25, 2024

Apply upstream refactorings to itinerary-page component and sub components

@andreashelms andreashelms requested a review from hbruch November 25, 2024 11:27
Copy link

@hbruch hbruch left a comment

Choose a reason for hiding this comment

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

Could you please check ItineraryListContainer? I think we should keep parkAndRide, Scooter and OnDemandTaxi.

Rest looks find to me, Thx!

Comment on lines 257 to 259
parkAndRide,
car,
onDemandTaxi,
Copy link

Choose a reason for hiding this comment

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

I dont't think we should remove these stadtnavi additions.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These props were not used in that component at all.

Comment on lines 311 to 314
scooter={scooter}
parkAndRide={parkAndRide}
car={car}
onDemandTaxi={onDemandTaxi}
Copy link

Choose a reason for hiding this comment

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

We should not remove stadtnavi additions

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Same here.

@andreashelms andreashelms force-pushed the refactor/itinerary-page-cleanup branch 2 times, most recently from 5834d83 to 0710ff8 Compare December 2, 2024 14:21
Base automatically changed from refactor/cleanup_itineraries to next December 2, 2024 14:23
@andreashelms andreashelms requested a review from hbruch December 2, 2024 17:56
@andreashelms andreashelms force-pushed the refactor/itinerary-page-cleanup branch 3 times, most recently from f8a6472 to 2c073e2 Compare December 4, 2024 16:34
@andreashelms andreashelms force-pushed the refactor/itinerary-page-cleanup branch from 2c073e2 to ec31263 Compare December 4, 2024 16:45
Copy link

@hbruch hbruch left a comment

Choose a reason for hiding this comment

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

Ok, we checked that scooters results are still provided. For OnDemand, even now herrenberg.stadtnavi.de shows no results, so this is unrelated to this change.

I approve this PR. Thx.

@andreashelms andreashelms force-pushed the refactor/itinerary-page-cleanup branch from fdf8904 to 61079cc Compare December 9, 2024 10:34
@andreashelms andreashelms force-pushed the refactor/itinerary-page-cleanup branch from 61079cc to fe61323 Compare December 9, 2024 10:36
@andreashelms andreashelms merged commit c416e3f into next Dec 9, 2024
5 checks passed
@andreashelms andreashelms deleted the refactor/itinerary-page-cleanup branch December 9, 2024 10:54
@hbruch hbruch mentioned this pull request Jan 22, 2025
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