Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

mariayord
Copy link
Contributor

@mariayord mariayord requested a review from renejeglinsky April 28, 2023 07:52
mariayord and others added 2 commits April 28, 2023 09:53
Co-authored-by: Samuel Brucksch <[email protected]>
Co-authored-by: Samuel Brucksch <[email protected]>
Comment on lines 305 to 309
For requests with an `$expand` query option on entities annotated with `@cds.localized: false`, the expanded properties are not translated.

```http
GET /BooksDetails?$expand=authors //> all fields from authors are non-localized defaults, if BooksDetails is annotated with `@cds.localized: false`
```
Copy link
Contributor

Choose a reason for hiding this comment

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

This limitation applies to CAP Node.js only. Please clarify this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't interpret this as a limitation...

Copy link
Contributor

@agoerler agoerler left a comment

Choose a reason for hiding this comment

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

Please clarify that this limitation applies to Node.js only.

@renejeglinsky renejeglinsky requested a review from agoerler May 2, 2023 07:47
@mariayord mariayord merged commit cff702e into main May 2, 2023
@mariayord mariayord deleted the read-localized-expand branch May 2, 2023 08: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