Skip to content

ERSAD: Update shelter or waiting room availability logic#1659

Merged
a-limyr merged 4 commits intoentur:masterfrom
solita-sabinaf:TIS-1235/update_shelter_or_waiting_room_availablity_logic
Feb 10, 2026
Merged

ERSAD: Update shelter or waiting room availability logic#1659
a-limyr merged 4 commits intoentur:masterfrom
solita-sabinaf:TIS-1235/update_shelter_or_waiting_room_availablity_logic

Conversation

@solita-sabinaf
Copy link
Collaborator

@solita-sabinaf solita-sabinaf commented Jan 23, 2026

Summary

Explain in one or two sentences what this PR achieves.

Issue

Update shelter or waiting room availability logic, so that it no longer checks the value of number of seats.

Screenshot 2026-01-28 at 17 17 35 Screenshot 2026-01-28 at 17 23 13

Closes #1658

Unit tests

Write a few words on how the new code is tested.

  • Unit tests for this already existed

@solita-sabinaf solita-sabinaf changed the title Update shelter or waiting room availability logic ERSAD: Update shelter or waiting room availability logic Jan 27, 2026
@solita-sabinaf solita-sabinaf force-pushed the TIS-1235/update_shelter_or_waiting_room_availablity_logic branch from 72e9996 to 6067bb1 Compare January 28, 2026 15:22
"not_assigned": "N/R",
"number_of_seats": "Nombre de places assises",
"seats": "Places assises disponibles",
"seats": "Places assises indisponibles",
Copy link
Contributor

Choose a reason for hiding this comment

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

duplicate key

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for catching it! The second was supposed to be seats_no

Copy link
Contributor

@testower testower left a comment

Choose a reason for hiding this comment

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

Just one issue with the french translation keys, otherwise lgtm

@solita-sabinaf
Copy link
Collaborator Author

The needed change got added 🚀

Copy link
Contributor

@a-limyr a-limyr left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@a-limyr a-limyr merged commit 034d379 into entur:master Feb 10, 2026
1 check passed
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.

Change the logic that determines whether a shelter or a waiting room are available

3 participants