Skip to content

chore: improve XML and JSON Schemas (DEV-449) - #180

Merged
gNahcab merged 10 commits into
mainfrom
wip/dev-449-improve-xml-json-schemas
Apr 25, 2022
Merged

gNahcab merged 10 commits into
mainfrom
wip/dev-449-improve-xml-json-schemas

Conversation

@gNahcab

@gNahcab gNahcab commented Apr 5, 2022 •

Copy link
Copy Markdown
Contributor

resolves DEV-449

@jnussbaum jnussbaum changed the title making element unique based on attribute name chore: improve XML and JSON Schemas (DEV-449) Apr 20, 2022
@jnussbaum jnussbaum self-assigned this Apr 20, 2022
@gNahcab gNahcab assigned gNahcab and unassigned jnussbaum Apr 25, 2022

@irinaschubert irinaschubert left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

Comment thread knora/dsplib/schemas/data.xsd Outdated
Comment on lines +203 to +212
<!-- boolean value type
<xs:complexType name="boolean_type">
<xs:sequence>
<xs:element name="text" type="xs:boolean" minOccurs="0" maxOccurs="1"/>
</xs:sequence>
<xs:attribute name="permissions" type="xs:NCName" use="required"/>
<xs:attribute name="comment" type="xs:string"/>
</xs:complexType>

</xs:complexType>-->

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can the commented out code be deleted?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, I'll delete it. Thanks for the review!

@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
0.0% 0.0% Duplication

@gNahcab
gNahcab merged commit 2c17b9d into main Apr 25, 2022
@gNahcab
gNahcab deleted the wip/dev-449-improve-xml-json-schemas branch April 25, 2022 16:13
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.

4 participants