Skip to content
This repository has been archived by the owner on Apr 5, 2022. It is now read-only.

Expose endpoint for accessing Profile URL instead of bytes and Flexibility on Comments, Posts and Likes #228

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BobbyJohansen
Copy link

No description provided.

@pivotal-issuemaster
Copy link

@BobbyJohansen Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@BobbyJohansen Thank you for signing the Contributor License Agreement!

Allow passing of extra query parameters
@BobbyJohansen BobbyJohansen changed the title Expose endpoint for accessing Profile URL instead of bytes Expose endpoint for accessing Profile URL instead of bytes and Flexibility on Comments, Posts and Likes Apr 11, 2017
@habuma
Copy link
Contributor

habuma commented Jun 16, 2017

@BobbyJohansen : I'd like to review and merge this PR, but it doesn't appear to include any tests (or adjustments to existing tests) to confirm that it does what is claimed. May I request that you add some tests to the PR? Having quality tests significantly eases the PR review process.

@BobbyJohansen
Copy link
Author

No problem I will update and ping you back when tests are in place.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants