Extract comments to a separate table

[licitdev]

Scope

What's changed:

• A new directus_comments table for comments

Potential Risks / Drawbacks

• Existing users reading into comments outside of the data studio will have to update their usage

Review Notes / Questions

• The comment field is now a required field
• Activity and revisions now exist for tracking
• Some additional logic is added in the app to continue supporting legacy comments that are added during the migration
• Should existing SDK & GQL remain unchanged until an upcoming version when the comment field is dropped from activity?
• Instead of removing the /activity/comment API, should an error message similar to the one in webhooks be added?
  

  directus/api/src/controllers/webhooks.ts

  Line 19 in d8c544e

  throw new (createError(ErrorCode.MethodNotAllowed, 'Webhooks are deprecated, use Flows instead', 405))();

---

Related to #17894

[changeset-bot]

🦋 Changeset detected

Latest commit: a7aaf23

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 23 packages

Name	Type
@directus/api	Major
@directus/sdk	Major
@directus/system-data	Minor
@directus/specs	Minor
@directus/types	Minor
@directus/app	Minor
directus	Patch
@directus/composables	Patch
@directus/utils	Patch
@directus/extensions-sdk	Patch
@directus/extensions	Patch
@directus/themes	Patch
@directus/validation	Patch
@directus/env	Patch
@directus/extensions-registry	Patch
@directus/memory	Patch
@directus/pressure	Patch
@directus/storage-driver-azure	Patch
@directus/storage-driver-cloudinary	Patch
@directus/storage-driver-gcs	Patch
@directus/storage-driver-s3	Patch
@directus/storage-driver-supabase	Patch
create-directus-extension	Patch

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

[br41nslug]

This PR will require some updated to the API reference docs and a breaking change note to cover the updated API.

Not sure if this is an issue in my setup or this PR but i was getting some infinite getSchema loop errors while attempting to run bootstrap until i disabled CACHE_SCHEMA

[br41nslug]

Do we have some form of timeout on migrations? if so then this loop may run into that for larger projects 🤔

[licitdev]

No, there isn't. CC @wrynegade