I think we definitely need to specify that 0 is ground floor, as that is variable between cultures. Adding examples would be a good idea, perhaps we could add it to the error message?
Of course! It's been a while since I dealt with Android XML layouts in depth.
Looks much nicer, thanks!
I think most users won't know how to correctly use a semicolon here. Is it common to have a business span multiple floors that can't be represented with a hyphenated range (eg 1-3, which my...
Could you please upload screenshots to make the review easier? Thanks for the contribution!
Thanks for the feedback! As you say, I'd expect that upon hearing the message the user would either return to the route (if it was a mistake) or disable TTS (if they have intentionally left the...
Missing assignment
Okay, lgtm, thanks!
The width of the shield doesn't account for the break. That's not great. But then again shields are not designed for this so a length limit should be fine.
I've read the diff and it looks sensible subject to my comments above. And low risk to merge as-is. It might be worth double-checking the external actions refs before merging just to make sure...
I think the Beta workflow will run on all commits to main, and I think that's wrong?
Ditto: v4