Skip to content
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

Add list_org_memberships_for_authenticated_user #357

Merged
merged 1 commit into from
May 12, 2023

Conversation

davidmhewitt
Copy link
Contributor

Allows us to get a list of the orgs the current user is a member of, and their role within that org (i.e. "admin", "member", etc)

@XAMPPRocky
Copy link
Owner

Thank you for your PR! It seems you need to run cargo fmt. 🙂

@davidmhewitt
Copy link
Contributor Author

@XAMPPRocky Oops, thanks! Should be good now.

@XAMPPRocky
Copy link
Owner

Thank you for that, would you mind squashing and pushing the commit? I normally wouldn't ask, but it seems something is borked with the workflow run, and it should be fixed by pushing a new commit.

@davidmhewitt davidmhewitt force-pushed the user_membership_orgs_api branch from e8227d2 to 7b4e3b3 Compare May 11, 2023 13:01
@davidmhewitt
Copy link
Contributor Author

Thank you for that, would you mind squashing and pushing the commit? I normally wouldn't ask, but it seems something is borked with the workflow run, and it should be fixed by pushing a new commit.

No problem. I'm never sure whether to squash commits when contributing to a new project. Some maintainers prefer the whole history to make the review easier and some prefer squashed commits. I've squashed this branch now.

@XAMPPRocky
Copy link
Owner

Thank you for your PR, and congrats on your first contribution! 🎉

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