Skip to content

Remove meeting feature#1062

Merged
SachaProbo merged 1 commit intomainfrom
SachaProbo/remove-meeting
Apr 20, 2026
Merged

Remove meeting feature#1062
SachaProbo merged 1 commit intomainfrom
SachaProbo/remove-meeting

Conversation

@SachaProbo
Copy link
Copy Markdown
Contributor

@SachaProbo SachaProbo commented Apr 16, 2026

Drop meetings and meeting_attendees tables, remove all meeting-related code across GraphQL, MCP, CLI, N8N, webhooks, frontend, and e2e tests.


Summary by cubic

Removed the Meetings feature across the app, including database tables, APIs, UI, automations, and webhooks. This simplifies the product and removes meeting-related dependencies.

  • Refactors

    • DB: added migration to drop meetings and meeting_attendees.
    • API/GraphQL: removed meeting types/resolvers/queries and node handling; dropped meeting webhook enum values; cleaned organization schema and policies; removed meeting service/actions and entity/audit types.
    • Console: deleted meetings pages/routes; removed the “Meetings” tab and copy from Context; updated WebhooksSettingsPage to remove meeting events; adjusted copy in PersonForm.
    • Integrations/Tooling/Tests: removed meeting tools from MCP; removed the n8n meeting resource and operations; updated prb webhook create examples; removed meeting helpers from test factories and e2e tests.
  • Migration

    • Run the new DB migration.
    • Remove all meeting GraphQL queries/mutations and types from clients.
    • Delete webhook subscriptions using meeting events and recreate with supported events (vendor/user/obligation).
    • Update n8n workflows to remove meeting nodes.
    • Remove MCP calls and tools that reference meetings.

Written for commit 050c280. Summary will update on new commits.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 45 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="pkg/cmd/webhook/create/create.go">

<violation number="1" location="pkg/cmd/webhook/create/create.go:69">
P2: The "all supported events" example is incomplete. Add OBLIGATION_CREATED, OBLIGATION_UPDATED, and OBLIGATION_DELETED to match the actual ValidEvents list.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment thread pkg/cmd/webhook/create/create.go Outdated
@SachaProbo SachaProbo force-pushed the SachaProbo/remove-meeting branch 6 times, most recently from bb1a509 to 050c280 Compare April 20, 2026 13:37
Drop meetings and meeting_attendees tables, remove all meeting-related
code across GraphQL, MCP, CLI, N8N, webhooks, frontend, and e2e tests.

Signed-off-by: Sacha Al Himdani <sacha@getprobo.com>
@SachaProbo SachaProbo force-pushed the SachaProbo/remove-meeting branch from 050c280 to 6c5c1fa Compare April 20, 2026 13:37
@SachaProbo SachaProbo merged commit 6c5c1fa into main Apr 20, 2026
17 checks passed
@SachaProbo SachaProbo deleted the SachaProbo/remove-meeting branch April 20, 2026 13:46
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.

1 participant