Skip to content

Adding Mapping table for OpenAPI to CDS Conversion( Import)#1848

Merged
renejeglinsky merged 5 commits into
mainfrom
openapi/mappingImportTable
Jul 18, 2025
Merged

Adding Mapping table for OpenAPI to CDS Conversion( Import)#1848
renejeglinsky merged 5 commits into
mainfrom
openapi/mappingImportTable

Conversation

@RoshniNaveenaS
Copy link
Copy Markdown
Contributor

No description provided.

@RoshniNaveenaS
Copy link
Copy Markdown
Contributor Author

@renejeglinsky can you comment if this table would be an useful information to have in OpenAPI page?

@renejeglinsky
Copy link
Copy Markdown
Contributor

I do think it's useful. Could you move it to the end where we also have this OData mapping table?

@renejeglinsky
Copy link
Copy Markdown
Contributor

@stewsk Do you think this table is helpful? Would you agree to move to the OData mapping table in this guide so that they are next to each other?

@renejeglinsky
Copy link
Copy Markdown
Contributor

Hi @stewsk ,
do you have an opinion here?

@stewsk
Copy link
Copy Markdown
Contributor

stewsk commented Jul 18, 2025

Hi @renejeglinsky , yes definitely helpful, and having it close to the OData table also seems reasonable

@renejeglinsky renejeglinsky requested a review from smahati as a code owner July 18, 2025 10:53
@renejeglinsky renejeglinsky merged commit b69a1f6 into main Jul 18, 2025
4 checks passed
@renejeglinsky renejeglinsky deleted the openapi/mappingImportTable branch July 18, 2025 11:14
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.

3 participants