Skip to content

feat: add mailing list frontend support (CM-1353) - #4479

Open
gaspergrom wants to merge 2 commits into
mainfrom
feat/CM-1353
Open

feat: add mailing list frontend support (CM-1353)#4479
gaspergrom wants to merge 2 commits into
mainfrom
feat/CM-1353

Conversation

@gaspergrom

Copy link
Copy Markdown
Contributor

Summary

  • Adds MAILING_LIST platform enum and identity configuration to enable mailing list activities and identities to display with platform icon and label across the frontend
  • Follows existing patterns (groupsio as reference): minimal code, SVG icon, no integration card enabled
  • Placeholder SVG icon included; final design asset must replace it before merge

JIRA

CM-1353 — Frontend: display support for mailing list integration

Notes

  • QA Status: Filters (PASS), Integrations page (PASS), Icon placeholder (PASS), Design asset blocker noted
  • Code quality: Minimal and follows established patterns
  • Quality gates: lint ✅, build ✅
  • Commit properly signed and formatted

Blocker: Icon asset from design must replace placeholder before merge (AC4)

Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Copilot AI balanced review requested due to automatic review settings August 14, 2026 23:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds frontend display support for mailing-list activities and member identities.

Changes:

  • Adds the mailinglist platform enum and identity configuration.
  • Registers mailing-list labels, links, filters, and icon rendering.
  • Adds a placeholder SVG asset pending final design.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
frontend/src/shared/modules/platform/types/Platform.ts Adds the mailing-list platform enum.
frontend/src/config/identities/mailinglist/config.ts Defines mailing-list identity display behavior.
frontend/src/config/identities/index.ts Registers the identity configuration.
frontend/src/assets/images/identities/mailing-list.svg Adds the placeholder platform icon.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread frontend/src/shared/modules/platform/types/Platform.ts
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Copilot AI review requested due to automatic review settings August 15, 2026 01:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated no new comments.

@gaspergrom
gaspergrom marked this pull request as ready for review August 15, 2026 01:55
@gaspergrom
gaspergrom requested a review from epipav August 15, 2026 01:55
@cursor

cursor Bot commented Aug 15, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Frontend-only identity/display wiring with no auth, API, or data-path changes; main follow-up is swapping the placeholder icon for the final design asset.

Overview
Adds Mailing List as a first-class identity platform so mailing-list activities and member/org identities can show the correct label, icon, and ordering alongside existing channels (pattern matches Groups.io).

Introduces Platform.MAILING_LIST (mailinglist), a new mailinglist IdentityConfig (member placeholder, activity links enabled), a placeholder envelope SVG, registration in lfIdentities, and MAILING_LIST placement in member and organization identity order configs (list, profile, suggestions).

Reviewed by Cursor Bugbot for commit 5264f43. Bugbot is set up for automated code reviews on this repo. Configure here.

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