Skip to content

Docs for new directus_comments collection#23641

Merged
rijkvanzanten merged 5 commits intoretentions-p1from
docs/directus_comments
Sep 13, 2024
Merged

Docs for new directus_comments collection#23641
rijkvanzanten merged 5 commits intoretentions-p1from
docs/directus_comments

Conversation

@phazonoverload
Copy link
Contributor

@phazonoverload phazonoverload commented Sep 11, 2024

Docs for #22295

Please do not merge until just before release, to provide the flexibility to release other minor changes before the next core release.

In review, please can the GraphQL implementations specifically be looked at closely as I've made some assumptions in them.

This page of the API reference also does not include 'real world' populated examples as with other endpoints, as they will not feature in the new docs and weren't too useful.

@changeset-bot

This comment was marked as off-topic.

@licitdev licitdev changed the base branch from main to retentions-p1 September 12, 2024 08:01
@rijkvanzanten
Copy link
Member

Please do not merge until just before release

We can merge this into revisions-p1 now so everything ships at the same time

@phazonoverload
Copy link
Contributor Author

Ah yes now we've changed the base branch that's all good once reviewed.

Copy link
Member

@rijkvanzanten rijkvanzanten left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@rijkvanzanten rijkvanzanten merged commit 7b0efc0 into retentions-p1 Sep 13, 2024
@rijkvanzanten rijkvanzanten deleted the docs/directus_comments branch September 13, 2024 08:56
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants