It seems like the usability of this module could be improved a lot if the embed widget was exposed as a block.

I've attached a patch; One note... The block is cached at a globally unless the requester email is configured to be set to the current user's email address in which case it's cached per user. It might be worth having no cache in this instance as it could well bloat the cache table a lot with a very low hit rate.

CommentFileSizeAuthor
freshdesk-expose_embed_widget_as_block-7_x.patch4.36 KBPQ
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

PQ created an issue. See original summary.

Perignon’s picture

Perignon CreditAttribution: Perignon commented

Thanks for the patch. I will take a look at this as soon as I can.

Perignon’s picture

Perignon CreditAttribution: Perignon commented

Bloating of cache isn't something many people worry about though. On high traffic sites you aren't using the SQL cache anyhow, you are using some NoSQL cache or Memcache.

Perignon’s picture

Perignon CreditAttribution: Perignon commented

This patch is going to have to be re-rolled. I applied your other patch first and now this will not apply. I applied the other patch first since it was a bug report.