Skip to content

ENG-1724 backend function: Remove members#1028

Merged
maparent merged 3 commits into
mainfrom
eng-1724-backend-function-remove-members
May 21, 2026
Merged

ENG-1724 backend function: Remove members#1028
maparent merged 3 commits into
mainfrom
eng-1724-backend-function-remove-members

Conversation

@maparent

@maparent maparent commented May 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

@linear-code

linear-code Bot commented May 13, 2026

Copy link
Copy Markdown

ENG-1724

@supabase

supabase Bot commented May 13, 2026 •
edited
Loading

Copy link
Copy Markdown

Updates to Preview Branch (eng-1724-backend-function-remove-members) ↗︎

Deployments Status Updated
Database ✅ Thu, 21 May 2026 13:52:58 UTC
Services ✅ Thu, 21 May 2026 13:52:58 UTC
APIs ✅ Thu, 21 May 2026 13:52:58 UTC

Tasks are run on every commit but only new migration files are pushed.
Close and reopen this PR if you want to apply changes from existing seed or migration files.

Tasks Status Updated
Configurations ✅ Thu, 21 May 2026 13:53:01 UTC
Migrations ✅ Thu, 21 May 2026 13:53:01 UTC
Seeding ✅ Thu, 21 May 2026 13:53:01 UTC
Edge Functions ✅ Thu, 21 May 2026 13:53:08 UTC

View logs for this Workflow Run ↗︎.
Learn more about Supabase for Git ↗︎.

@maparent maparent marked this pull request as draft May 13, 2026 01:43
@graphite-app

graphite-app Bot commented May 13, 2026

Copy link
Copy Markdown
Contributor

PR size/scope check

This PR is over our review-size guideline.

  • Recommended: ~200 lines changed
  • Acceptable limit: up to 400 lines when well-scoped/self-contained
  • Preferred file count: fewer than 5 files

Please split this into smaller PRs unless there is a clear reason the changes need to land together.

If keeping it as one PR, please add a brief justification covering:

  • What single problem this PR solves
  • Why the files/changes are coupled

@maparent maparent force-pushed the eng-1724-backend-function-remove-members branch 4 times, most recently from ffb4cc4 to f0eef7c Compare May 17, 2026 19:36
@maparent maparent changed the base branch from main to eng-1758-vitest-infrastructure May 17, 2026 19:39
@maparent maparent force-pushed the eng-1758-vitest-infrastructure branch from 4a8f9bd to 8fde609 Compare May 18, 2026 11:20
@maparent maparent force-pushed the eng-1724-backend-function-remove-members branch from f0eef7c to 35d7c2b Compare May 18, 2026 11:21
@maparent maparent force-pushed the eng-1758-vitest-infrastructure branch from 8fde609 to a5c951e Compare May 18, 2026 13:33
@maparent maparent force-pushed the eng-1724-backend-function-remove-members branch from 35d7c2b to d2ed2d7 Compare May 18, 2026 13:47
@maparent maparent force-pushed the eng-1758-vitest-infrastructure branch from a5c951e to 66a8fe4 Compare May 18, 2026 16:01
Base automatically changed from eng-1758-vitest-infrastructure to main May 18, 2026 19:39
@maparent maparent force-pushed the eng-1724-backend-function-remove-members branch 3 times, most recently from 02cf242 to 5cc41a9 Compare May 20, 2026 19:00
@maparent maparent changed the base branch from main to eng-1702-invite-new-members-in-the-group May 20, 2026 19:01
@maparent maparent requested a review from mdroidian May 20, 2026 19:27
@maparent maparent marked this pull request as ready for review May 20, 2026 19:27
graphite-app[bot]
graphite-app Bot reviewed May 20, 2026
View reviewed changes
Comment thread apps/website/test/integration/leave_group.test.ts Outdated
@maparent maparent force-pushed the eng-1724-backend-function-remove-members branch from 1066da0 to 8809216 Compare May 20, 2026 19:53
@mdroidian

mdroidian commented May 20, 2026

Copy link
Copy Markdown
Member

@codex review

chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed May 20, 2026
View reviewed changes

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 88092169c9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/website/app/utils/supabase/account.ts
mdroidian
mdroidian approved these changes May 21, 2026
View reviewed changes

@mdroidian mdroidian left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also see graphite/codex comments

Comment thread apps/website/test/integration/leave_group.test.ts
@maparent maparent force-pushed the eng-1702-invite-new-members-in-the-group branch from 94dd945 to 3843ceb Compare May 21, 2026 12:58
Base automatically changed from eng-1702-invite-new-members-in-the-group to main May 21, 2026 13:27
@maparent maparent force-pushed the eng-1724-backend-function-remove-members branch from 8809216 to 945d4c5 Compare May 21, 2026 13:28
@maparent maparent merged commit 9568d34 into main May 21, 2026
8 checks passed
@maparent maparent deleted the eng-1724-backend-function-remove-members branch May 21, 2026 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@graphite-app graphite-app[bot] graphite-app[bot] left review comments
@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments
@mdroidian mdroidian mdroidian approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@maparent @mdroidian