Skip to content

Conversation

@achou11
Copy link
Member

@achou11 achou11 commented Oct 28, 2025

Integrates @comapeo/core@5.1.1. 5.1.0 introduces APIs related to removing project members and emitting events for when your own role changes.

This PR introduces the following:

  • a write hook called useRemoveMember() to remove a project member
  • a listener hook called useSetUpProjectOwnRoleChangeListener() to make sure that read hooks that are affected by the emitted role change events happening in the background are invalidated.

I'm a little bit unsure about the need for the listener hook, but I figured it would be similar to the invites listener hook we use.

@awana-lockfile-bot
Copy link

awana-lockfile-bot bot commented Oct 28, 2025

package-lock.json changes

Click to toggle table visibility
Name Status Previous Current
@comapeo/core UPDATED 5.0.0 5.1.1
@comapeo/schema UPDATED 2.1.1 2.2.0

@socket-security
Copy link

socket-security bot commented Oct 28, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​comapeo/​core@​5.0.0 ⏵ 5.1.17910084 +197 +1100
Updated@​comapeo/​schema@​2.1.1 ⏵ 2.2.084 +410093 +194 +2100

View full report

Copy link

@RangerMauve RangerMauve left a comment

Choose a reason for hiding this comment

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

Small comments else LGTM

Copy link
Contributor

@ErikSin ErikSin left a comment

Choose a reason for hiding this comment

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

Code looks straight forward and works if we are following the same pattern as the invite listener!

@achou11 achou11 merged commit c005505 into main Oct 30, 2025
9 checks passed
@achou11 achou11 deleted the ac/core-5.1.0 branch October 30, 2025 18:07
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.

3 participants