Skip to content

(feat): Embedded Comments first stab

Mark Harding requested to merge feat/embedded-comments into master

Ticket(s) / Related Merge Requests

minds#4533 (closed)

Summary of Changes

Introduces new graphql endpoints to support an embedded comments api. Also changes our session and XSRF cookie to be SameSite=None.

Testing Considerations

Deployment Considerations

CREATE TABLE IF NOT EXISTS  minds_embedded_comments_activity_map (
    tenant_id int DEFAULT -1,
    user_guid bigint NOT NULL,
    url varchar(256) NOT NULL,
    activity_guid bigint NOT NULL,
    PRIMARY KEY (tenant_id, user_guid, url)
);

Regression Scope

Platform Affected (web, mobile, etc)

Developer Testing Completed

Screenshots / Screen Recording

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Mark Harding

Merge request reports