Skip to content

Conversation

@joshlarson
Copy link
Contributor

@joshlarson joshlarson commented Dec 22, 2025

@joshlarson joshlarson force-pushed the jdl/sf2.0/ud/more-bus-and-cr-trips branch from 57c7683 to 6eb2852 Compare December 22, 2025 18:16
@joshlarson joshlarson marked this pull request as ready for review December 22, 2025 19:47
@joshlarson joshlarson requested a review from a team as a code owner December 22, 2025 19:47
@joshlarson joshlarson added the do not merge ⏳ it's waiting on something label Dec 22, 2025
@joshlarson
Copy link
Contributor Author

I marked it as "Do not merge" because it's branched off of #2850, so that one should be merged first.

Base automatically changed from jdl/sf2.0/ud/bus-interleave-schedule-and-prediction-data to main December 23, 2025 20:42
@joshlarson joshlarson force-pushed the jdl/sf2.0/ud/more-bus-and-cr-trips branch from 6eb2852 to 673c5aa Compare December 23, 2025 20:47
@joshlarson joshlarson removed the do not merge ⏳ it's waiting on something label Dec 23, 2025
@joshlarson joshlarson enabled auto-merge (squash) December 23, 2025 20:54
<.remaining_service
:if={departures = @departures.ok? && @departures.result}
route_type={@route.type}
:if={departures = @stop && @departures.ok? && @departures.result}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Was departures = @stop a mistake??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Noooo - it's probably not the best way to do it, but I was assigning departures to that I could use it in end_of_service(departures).

@joshlarson joshlarson merged commit 95eed21 into main Dec 30, 2025
17 checks passed
@joshlarson joshlarson deleted the jdl/sf2.0/ud/more-bus-and-cr-trips branch December 30, 2025 17:08
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.

3 participants