MMM-Cocktails - Images now showing up?

Mykle1

I will take a look at this tonight. Sorry for the delay. Work and grandchildren keeping me busy.

HYKGOML

No apologies needed at ALL. Those things always are more important.
… and don’t skin me over the fact I also put in a defect on Git… Was not sure best place to post this.

I look forward to to the process as I have theories on whats wrong and what the fix might be. But I gave it a go did not have success.

Now go back to having a life!

Mykle1

@hykgoml

Thanks for bringing that to my attention. I don’t always check for issues on the hub. Evidently, the api changed a bit but the fix was simple. See line 65 and 66 of the MMM-Cocktails.js file if you’re curious. I’ve had it running for an hour now without incident.

I’ll upload the update to the hub. You can do a git pull in the MMM-Cocktails directory for the update. Let me know how you make out.

EDIT:

Updated on github

cowboysdude

@hykgoml Generally fixes are simple for the developer so if you have a problem just let them know ;) LOTS easier then trying to figure it out…you may really break something in the process ;)

HYKGOML

@mykle1
(reads line 65-66)
Ohhh ok I see what you did… Cool thanks! (not a java programmer) so the syntax is a bit new for me.

Thanks again!

TLDR; Yep works great!

Cheers!

Mykle1

@hykgoml

Great!