Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Update trace location attributes validation #2732

Merged

Conversation

jurajkusnier
Copy link
Contributor

Update trace location attributes validation according to spec changes:

  • Bezeichnung (description) may not be empty, may not contain any line breaks, and may not have more than 100 characters after removing any leading or trailing whitespace
  • Ort (address) may not be empty, may not contain any line breaks, and may not have more than 100 characters after removing any leading or trailing whitespace
  • Beginn (start) must be before Ende (end) if the type is an event; both must be unset if the type is a location.
  • Typische Aufenhaltsdauer (default check-in length) must be greater than zero if the type is a location and must be great than or equal zero if the type is an event (zero meaning no value set)

@jurajkusnier jurajkusnier added maintainers Tag pull requests created by maintainers prio PRs to review first. labels Apr 1, 2021
@jurajkusnier jurajkusnier added this to the 2.0.0 milestone Apr 1, 2021
@jurajkusnier jurajkusnier requested a review from a team April 1, 2021 11:11
@axelherbstreith axelherbstreith self-assigned this Apr 1, 2021
@AlexanderAlferov AlexanderAlferov self-assigned this Apr 1, 2021
@BMItr BMItr self-assigned this Apr 1, 2021
@sonarcloud
Copy link

sonarcloud bot commented Apr 1, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

87.5% 87.5% Coverage
0.0% 0.0% Duplication

@jurajkusnier jurajkusnier merged commit 595087d into release/2.0.x Apr 1, 2021
@jurajkusnier jurajkusnier deleted the dev/update_trace_location_attributes_validation branch April 1, 2021 13:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
maintainers Tag pull requests created by maintainers prio PRs to review first.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants