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

fix: fix routes with no containers associated #263

Merged
merged 10 commits into from
Jul 27, 2024

Conversation

joaotomaspinheiro
Copy link
Collaborator

Summary

Prevent routes from being created and updated without containers associated in the UI.

Changes

  • Update validation logic on the route edit form
  • Remove the disabled logic of the save button inside the select containers modal, since it's now validated on the form
  • Add custom message on the map page of a route if it does not have any containers associated with it

@joaotomaspinheiro joaotomaspinheiro added bug Something isn't working web app Work associated with the web app labels Jul 25, 2024
@joaotomaspinheiro joaotomaspinheiro self-assigned this Jul 25, 2024
Base automatically changed from 261-tailwind-css-migration to main July 27, 2024 15:08
@joaotomaspinheiro joaotomaspinheiro merged commit 88be493 into main Jul 27, 2024
3 checks passed
@joaotomaspinheiro joaotomaspinheiro deleted the bugfix/route-containers branch July 27, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working web app Work associated with the web app
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants