Slack app /looks cmd gives repeats same preview

leighmcculloch / looks.wtf

A collection of looks. (▰˘◡˘▰)

https://looks.wtf

102 stars 21 forks source link

Slack app /looks cmd gives repeats same preview #15

Closed one000mph closed 1 year ago

one000mph commented 5 years ago

Hi I just installed your slack app (which is so great btw) and noticed a bug which causes the same ascii to repeat multiple times as shown here

I understand it can be difficult to find time to update open source stuff but wanted to bring it to your attention. I might be able to submit a PR to fix this in a couple weeks

leighmcculloch commented 5 years ago

Oh no! I've never seen this, amazing it took this long for this to happen. If you'd be up for taking a stab at a fix you're welcome!

ronkagan commented 4 years ago

/looks does not appear to be supported in Slack anymore?

leighmcculloch commented 4 years ago

@ronkagan Are you find the same issue now? The service appears to be working correctly over here. Is there any chance the Slack admin remove it from your Slack?

ronkagan commented 4 years ago

Ahhh, I had gotten the admin to approve the installation. I had to install it though:

Now...

Fab!

leighmcculloch commented 1 year ago

This issue is fixed in the latest deployment because the way looks are randomized, they can't be repeated.