Hey there,

I just updated to 6.x-5.x-dev, as previous version didn't work. Now I'm wondering where to set my API Key for the Thumbalizr service. Is this not implemented yet or am I plain blind?

Comments

daniel wentsch’s picture

Okay, found it by looking up wss_thumbalizr.module :)
For everybody else who might be searching:
admin/settings/website_screenshot/thumbalizr
Seems you have to be admin user, since there's no permission you can set yet.

PS: after visiting the URL mentioned above once, the menu items for Bulga and Thumbalizr suddenly appeared.

uccio’s picture

Assigned: Unassigned » uccio
Status: Active » Closed (fixed)

thanks for the feedback!

ice5nake’s picture

Shouldn't this be re-opened and filed as a bug or at the very least a feature request?

This behavior seems very deep in dev mode and not of release quality.

ice5nake’s picture

Category: support » bug
Status: Closed (fixed) » Active

This indeed seems like a bug because in the hook_menu implementations for the engines there is reference to a permission "administer screenshot". However there is no implementations of hook_perm that create this permission.

ice5nake’s picture

Status: Active » Needs review
StatusFileSize
new1.51 KB

Patch made to add the "administer screenshot" permission so that the settings pages for the API key would appear.

uccio’s picture

Status: Needs review » Closed (fixed)

ice5nake,

thanks for the patch.
I fix the path privilegies problem and now you can download patched version.

thanks!!

josernitos’s picture

Is this fix only for 6.x? Or it's also to 7.x?

I'm trying to use thumbalizr, but i don't see it.

Thanks

NathanM’s picture

Version: 6.x-5.x-dev » 7.x-1.x-dev
Status: Closed (fixed) » Active

Yup, also not seeing any thumbalizr support in 7.x

meecect’s picture

yep, same here. Not seeing any setting for any services on 7.x

puravida’s picture

Title: Where can I provide the API-Key to thumbalizr? » Missing permissions hide Admin UI by default
Issue summary: View changes

Changing the title to be a little more specific, since there are duplicate threads and this one has the patch for at least 6.x branch. No response on whether it was ported to the 7.x branch or not. Will have to investigate.

-Brandon