Skip to content

Conversation

@Jeff-Meadows
Copy link
Contributor

Fixes #110

@boxcla
Copy link

boxcla commented Feb 10, 2016

Verified that @Jeff-Meadows has signed the CLA. Thanks for the pull request!

):
"""
Get a shared link download url for the file with the given access permissions.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a sentence or a link to explain the difference between shared link and direct url? I guess it might be confusing to a few users.

@uvdn7
Copy link
Contributor

uvdn7 commented Feb 11, 2016

👍 a small comment on docstring.

Jeff-Meadows added a commit that referenced this pull request Feb 11, 2016
Expose file.get_shared_link_download_url method.
@Jeff-Meadows Jeff-Meadows merged commit fb4f243 into box:master Feb 11, 2016
@Jeff-Meadows Jeff-Meadows deleted the slinkurl branch February 11, 2016 21:39
@nehalecky
Copy link

Hey wow! That was super fast, thanks all!

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.

4 participants