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

[WIP]: Self identity fixes #1448

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[WIP]: Self identity fixes #1448

wants to merge 5 commits into from

Conversation

tkurki
Copy link
Member

@tkurki tkurki commented May 29, 2022

Fixes #1446.
See #1447.

If the user never saves settings from the UI the uuid
was not saved and a new one would be generated on every
server restart.
@tkurki tkurki added the feature label May 29, 2022
Validate mmsi to be 9 digits and uuid to have the right format.
Require either mmsi or uuid to be set.
Validation added to 'vessel' endpoint that admin UI
uses so that admin UI shows the error message.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Uuid should be persisted even without running setup or adding settings
1 participant