Skip to content

Conversation

@sthesse
Copy link
Contributor

@sthesse sthesse commented Oct 21, 2024

... based on experts review.

... based on experts review.
sthesse and others added 3 commits October 23, 2024 09:52
Co-authored-by: Heiko Theißen <heiko.theissen@sap.com>
Co-authored-by: Heiko Theißen <heiko.theissen@sap.com>
Co-authored-by: Heiko Theißen <heiko.theissen@sap.com>
ralfhandl
ralfhandl previously approved these changes Oct 23, 2024
"type": "boolean",
"default": true,
"description": "Property contains potentially personal data. Properties annotated with x-sap-dpp-field-semantics need not be additionally annotated with this extension."
"description": "Property contains potentially personal data. Properties annotated with x-sap-dpp-field-semantics need not be additionally annotated with this extension. Personal data describes any information which is related to an identified or identifiable natural person (data subject). An identifiable person is one who can be identified, directly or indirectly, in particular by a reference to an identifier such as a name, an identification number, location data, an online identifier, or to one or more factors specific to the physical, physiological, genetic, mental, economic, cultural, or social identity of that natural person."
Copy link
Member

Choose a reason for hiding this comment

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

Please generate SAP schemas ! run npm run generate and commit those files to this PR. Thx.

Copy link
Contributor

Choose a reason for hiding this comment

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

@pavelkornev How about automating that step with a GitHub action?

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, i think we should generate markdown files in general and have just one place where we define things, not 2.

@pavelkornev pavelkornev merged commit 1d063d3 into SAP:main Nov 18, 2024
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