Skip to content

Fix UpsertGroup roles #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ludydoo
Copy link

@ludydoo ludydoo commented Apr 4, 2025

Addresses #80

@ludydoo ludydoo changed the title fix-user-roles Fix UpsertGroup roles Apr 4, 2025
@chvck
Copy link
Contributor

chvck commented Apr 7, 2025

Hi @ludydoo thanks for your contribution! This looks good to me.

We're currently using Gerrit for reviews so, unfortunately, you'll need to sign our CLA there. There's a bit of information on how to do that at https://github.com/couchbase/couchbase-jvm-clients/blob/master/CONTRIBUTING.md#pull-requests (note: the bot that it mentions no longer exists). If you can sign the CLA then I'll pull the change over to gerrit and get it merged.

@chvck
Copy link
Contributor

chvck commented Apr 22, 2025

Hi @ludydoo were you able to take a look at signing our CLA? We can't see that you have.

@ludydoo
Copy link
Author

ludydoo commented May 14, 2025

Hey @chvck , yes I signed the CLA just now :)

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