Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: JSON:API refactor #472

Merged
merged 3 commits into from
May 3, 2024
Merged

docs: JSON:API refactor #472

merged 3 commits into from
May 3, 2024

Conversation

SychO9
Copy link
Member

@SychO9 SychO9 commented Mar 14, 2024

Documents breaking changes for the upgrade guide and links to the new documentation for API resources.

It adds links to tags extension from 1.x and the one from 2.x as an example for before and after.

@SychO9 SychO9 mentioned this pull request Mar 14, 2024
2 tasks
docs/extend/update-2_0.md Show resolved Hide resolved
static/en/img/api_flowchart.svg Outdated Show resolved Hide resolved
docs/extend/api.md Outdated Show resolved Hide resolved
docs/extend/api.md Show resolved Hide resolved
docs/extend/api.md Outdated Show resolved Hide resolved
docs/extend/api.md Outdated Show resolved Hide resolved
docs/extend/api.md Outdated Show resolved Hide resolved
## Serializers
### Inclusion & Linkage

You can mark a relationship as includable through the `includable` method. This means that the relationship can be included in the API response. You can also use the `withLinkage` and `withoutLinkage` methods to determine whether the relationship ID(s) should be included in the API response (`ToMany` relationships are not linked by default contrary to `ToOne` relationships).
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

When would you have non-includable relationships?

Copy link
Member Author

Choose a reason for hiding this comment

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

When you only need a relationship to be writable perhaps. Though not a common use case, it remains valid.

Copy link
Sponsor Member

Choose a reason for hiding this comment

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

Might be worth adding a short tip about this

docs/extend/api.md Show resolved Hide resolved
docs/extend/api.md Outdated Show resolved Hide resolved
SychO9 and others added 2 commits March 27, 2024 18:42
@SychO9 SychO9 requested a review from askvortsov1 March 27, 2024 20:17
@SychO9 SychO9 merged commit 8de54dd into 2.x May 3, 2024
1 check passed
@SychO9 SychO9 deleted the sm/json-api-server branch May 3, 2024 07:41
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.

2 participants