Skip to content

Conversation

@Aaylor
Copy link
Collaborator

@Aaylor Aaylor commented Jun 3, 2019

In order to list the current user organizations, and for which the user has granted the OAuth application the acces to the organizations, we must use the API /user/orgs.

It is different from /users/:username/orgs: it returns only public organizations, which the OAuth application (via the token) may not have been granted access.

@avsm
Copy link
Member

avsm commented Jun 3, 2019

Thanks very much! CI appears broken due to an issue that's nothing to do with your contribution. I'll fix that up separately and then merge/release your PR :-)

@avsm
Copy link
Member

avsm commented Jun 3, 2019

I've pushed a merge to your branch that should now pass CI

@avsm avsm merged commit c16b4ac into mirage:master Jun 4, 2019
avsm added a commit to avsm/opam-repository that referenced this pull request Jun 4, 2019
CHANGES:

- Add the interface for `/user/orgs` (mirage/ocaml-github#222 @Aaylor)
- Switch to dune-release instead of topkg (mirage/ocaml-github#224 @avsm)
- Do not use deprecated `Yojson.Safe.json` (mirage/ocaml-github#224 @avsm)
- Support lambda-term/zed 2.0.0 interfaces (mirage/ocaml-github#224 @avsm)
- Use wrapped `js_of_ocaml` 3.4.0 interfaces (@avsm)
@avsm
Copy link
Member

avsm commented Jun 4, 2019

Released into opam. Thanks for the 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