Extract comments to a separate table#22295
Merged
DanielBiegler merged 69 commits intoretentions-p1from Sep 6, 2024
Merged
Conversation
🦋 Changeset detectedLatest commit: 47791b9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 23 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
licitdev
changed the title
Member
|
LGTM 🚀 Some things to note:
P.S. Left off approval as I was not sure if |
Co-authored-by: daedalus <[email protected]>
Co-authored-by: Daniel Biegler <[email protected]>
DanielBiegler
approved these changes
Sep 6, 2024
Contributor
DanielBiegler
left a comment
DanielBiegler
left a comment
There was a problem hiding this comment.
After the latest rounds of changes it worked for me, lets move on to the testing branch
ComfortablyCoding
added a commit
that referenced
this pull request
Oct 16, 2024
* Extract comments to a separate table (#22295) * Add directus_comments migration * Add comments controller and service * Remove from activity * Update system-data and types * Refactor app with new endpoints * Expose service * Update app minimal permissions * Add collection translation * Define relations * Allow comment creation only if there's item read access * Patch for MSSQL double constraints issue * Fix users service test * Add sdk support * Update specs * Fix formatting * Fix specs error * Patch whoopsie * Remove obsolete GraphQL mutations * Update required fields * Remove unused vars * Allow edit and delete of legacy activity comments * Remove legacy comments from SDK * Add changeset * Batch upwards migration * Update SDK to use keysOrQuery Co-authored-by: Brainslug <[email protected]> * Update implementation for keysOrQuery * Remove singleton check * Update SDK to use keysOrQuery 2 Co-authored-by: Brainslug <[email protected]> * Update keysOrQuery typedoc * Fix import * Update migration timestamp * fixed import * Update api/src/utils/get-service.ts * utilize chunk processing in migration * formatting * only services extended from itemservice should be added * remove redundant checks from comment header * update comment service to v11 permission format * specify missing required fields * Mock comments in users test * Simplify migration and update date * WIP legacy access * Optimise imports * WIP app cleanup * Update loadUserPreviews typing Co-authored-by: Daniel Biegler <[email protected]> * Read legacy comments * Parse using comments service * Perform migration directly * Fix legacy app sort query which uses id * Migrate legacy comments in mutations * Reduce api semver * Update app recommended permissions * Recommend updating of comment only * replace hardcoded type with existing one * Allow users to update or delete their own comments Co-authored-by: daedalus <[email protected]> * Skip further access validation for non-existent collections * Check if collection exists before the admin check Co-authored-by: daedalus <[email protected]> * Fix incorrect legacy check Co-authored-by: Daniel Biegler <[email protected]> * Fix merging of count when db returns count as string type * Remove unused import --------- Co-authored-by: Brainslug <[email protected]> Co-authored-by: Pascal Jufer <[email protected]> Co-authored-by: Brainslug <[email protected]> Co-authored-by: daedalus <[email protected]> Co-authored-by: Daniel Biegler <[email protected]> * Consolidate content versioning (#22413) * Add migration * Use the new delta field * Add cast-json flag * Fix typing * Fetch existing deltas if version created during migration * Add changeset * Add version delta field into sdk schema * Update migration timestamp * Update versions.save() to return finalVersionDelta Co-authored-by: Pascal Jufer <[email protected]> * Sort on DB level * Update migration date * Disallow passing delta via create/update * Update docs & specs * Fix save response * Remove unnecessary access check Already checked by the subsequent itemsService.readOne call * Update changeset * Don't require update perms on versions for save * Optimize validateCreateData * update to new validateAccess * Update docs/reference/system/versions.md * Remove migration of delta * Rename to legacy * Add missed changes for Remove migration of delta in 2e2f50f * Update docs/reference/system/versions.md --------- Co-authored-by: Pascal Jufer <[email protected]> Co-authored-by: daedalus <[email protected]> * Update migration dates * api change should be major for versioning * Remove comment paths from activity reference * Added comments reference * Added directus_comments to table of system collections * The linter demands newline * Revert function renaming for patch semver * Use transaction in down migration for comments (#23715) --------- Co-authored-by: Brainslug <[email protected]> Co-authored-by: Pascal Jufer <[email protected]> Co-authored-by: Brainslug <[email protected]> Co-authored-by: daedalus <[email protected]> Co-authored-by: Daniel Biegler <[email protected]> Co-authored-by: Kevin Lewis <[email protected]> Co-authored-by: Rijk van Zanten <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
What's changed:
directus_commentstable for commentsPotential Risks / Drawbacks
Review Notes / Questions
directus_activitytodirectus_commentsdirectus_commentswhen it is updated or deletedmainand accessing it from the dev api server in this PRRelated to #17894
Closes SER-255