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

Convert dropbox embedded image page URLs to direct image links #67

Closed
JoeGermuska opened this issue Aug 19, 2015 · 2 comments · Fixed by #87
Closed

Convert dropbox embedded image page URLs to direct image links #67

JoeGermuska opened this issue Aug 19, 2015 · 2 comments · Fixed by #87

Comments

@JoeGermuska
Copy link
Member

As with #64, this is probably better handled in the authoring tool rather than create a special Image subclass.

Given a URL like
https://www.dropbox.com/s/yuy2dxj6pxz9jf8/cole.jpg?dl=0
convert the domain in the URL as such:
https://dl.dropboxusercontent.com/s/yuy2dxj6pxz9jf8/cole.jpg?dl=0

@duner
Copy link
Contributor

duner commented Aug 19, 2015

We could do that. Another thought I've had for the Dropbox part is to
enable their "chooser" code which can return the direct link to a file and
also is maybe a better UI?
https://www.dropbox.com/developers/dropins/chooser/js

On Wed, Aug 19, 2015 at 6:54 PM, Joe Germuska notifications@github.com
wrote:

As with #64 #64, this is
probably better handled in the authoring tool rather than create a special
Image subclass.

Given a URL like
https://www.dropbox.com/s/yuy2dxj6pxz9jf8/cole.jpg?dl=0
convert the domain in the URL as such:
https://dl.dropboxusercontent.com/s/yuy2dxj6pxz9jf8/cole.jpg?dl=0


Reply to this email directly or view it on GitHub
#67.

duner.me
@asduner http://twitter.com/asduner

@JoeGermuska
Copy link
Member Author

Let's do the simple thing now, and if we want to add the chooser parts, that can be a separate issue/initiative

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 a pull request may close this issue.

2 participants