Skip to content
This repository was archived by the owner on Aug 15, 2023. It is now read-only.

Conversation

@robwilkerson
Copy link

Addresses issue #14.

Sometimes you need to pull data from repos that aren't owned by you, but rather by a team that you belong to. This pull request allows an owner/username to be passed to several APIs and the API will do its thing as long as the authenticated user has access to the requested repository.

I didn't add anything to the tests, but I did run them and everything still seems okay.

Sometimes you need to pull data from repos that arne't owned by you, but
rather by a team that you belong to.
This change uses the response.content value rather than response.text
because binary content is corrupted when writing the text to the zip
file during an archive operation.
@fala
Copy link

fala commented Mar 31, 2015

This is exactly what I wanted to fix now. Please merge!

@robwilkerson
Copy link
Author

@fala I don't know when/if this will ever get merged, but I have a fork that you can install in the interim, if you're desperate:

pip install git+git://github.com/robwilkerson/BitBucket-api.git

This is what I had to do to get it working for me while I wait on feedback on the PR.

This is follow up to commit 462c1be, solving the problem in the issue.py file.
@fala
Copy link

fala commented Mar 31, 2015

Actually I need mostly issue.py file - so I've fixed it in the same manner. There it goes in the pull request, so you can patch your solution so it covers that file as well.

robwilkerson and others added 16 commits March 31, 2015 17:04
Thanks, @fala. I appreciate it.
issue_comment from issues for any accessible owner.
Adding `Repository.team()` method.

Returns repositories to which the user has access via teams (public or private). Thanks, @reece.
Adds some v2 functionality around Pull Requests. Thanks, @iashwash.
Get diff of a pull request.
Adding creation of repos through team with api v2.
This ended up going to the v2 URL for old scripts that weren't ready for
that.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants