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

s/tumbor/thumbor/ #18

Merged
merged 1 commit into from Feb 20, 2018
Merged

s/tumbor/thumbor/ #18

merged 1 commit into from Feb 20, 2018

Conversation

kylemarsh
Copy link
Contributor

Fix typo added when moving from if-else to switch.

Fix typo added when moving from if-else to switch.
@kylemarsh
Copy link
Contributor Author

Aside from this change, does anyone have a sample thumbor config file for using with 50mm? I haven't found any documentation for it yet (maybe I'm just missing it?) and I haven't been able to make it work yet: the URLs 50mm generates seem to be invalid (at least given the stock configuration). For instance:

https://localhost:8888/GOOP/800x0/smart/prefix/to/photo.jpg
as opposed to
http://localhost:8888/GOOP/800x0/smart/http://bucket.objectstore.com/prefix/to/photo.jpg

(also, it's generating HTTPS links by default and I'm not sure if there's a way to disable that or if I need to actually get a legitimate certificate for my thumbor service)

Any advice? Since the individual image pages exist now, I've got a change to the CSS to make the album page more of a set of thumbnails, but I want to make sure it works with resizing services before I submit a pull request for it. In deference to the desire for simplicity, I also might see about putting something into the configuration to specify whether an album page shows thumbnails or full-sized images. For the moment you can get a preview of the responsive album page here: http://photos.kmarsh.net/bm2017/

@arahayrabedian
Copy link
Collaborator

Hi @kylemarsh , thanks for catching that typo!

I'll have to get back to you on your questions - wrote the thumbor implementation a while back and my morning just started.

I can't remember why HTTPS was enforced, but I'll look in to if an override is feasible without compromising anything, that said, huge fan of let's encrypt for free certs (plus caddy web server to make life even easier!)

@arahayrabedian arahayrabedian merged commit 2d6dc0f into agile-leaf:master Feb 20, 2018
@arahayrabedian
Copy link
Collaborator

@kylemarsh can you do me a favour and open an issue with basically the same text pasted in so I can get back to you later without making a mess here?

@kylemarsh
Copy link
Contributor Author

Sure thing! Thanks for the speedy response.

@kylemarsh
Copy link
Contributor Author

Done. I actually split it into two issues; #19 is about thumbor and #20 is seeding a discussion about possibly incorporating the responsive album overview pages I made for my personal site.

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.

None yet

2 participants