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

Auto slack uploader fails #284

Open
Darth-Arminius opened this issue Dec 18, 2018 · 5 comments
Open

Auto slack uploader fails #284

Darth-Arminius opened this issue Dec 18, 2018 · 5 comments

Comments

@Darth-Arminius
Copy link
Contributor

Darth-Arminius commented Dec 18, 2018

Not sure if this has already been mentioned, already known or if it is specifically a problem with emojipacks (it probably is but I felt it was worth mentioning it here).

I followed the instructions to start the auto upload:

First I ran yarn install

Then yarn build which initially had test failure problems but that seems to have been fixed.

Then I ran emojipacks
I entered my slack details and the absolute path to the parrotparty.yaml file.

Then I get this error:

Got tokens
Logged in
Uh oh! Error: Login error: could not get emoji upload crumb for https://[my-subdomain].slack.com
(node:11208) UnhandledPromiseRejectionWarning: Error: Login error: could not get emoji upload crumb for https://[my-subdomain].slack.com
    at Slack.emoji ([my-path]\npm\node_modules\emojipacks\lib\slack.js:162:34)
    at Slack.emoji.next (<anonymous>)
    at onFulfilled ([my-path]\\npm\node_modules\emojipacks\node_modules\co\index.js:65:19)
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:11208) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:11208) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

This most probably has something to do with emojipacks and it seems many people have voiced their errors there too. It seems that what this issue is about is the most likely culprit. Not sure if/when that will be fixed or how long.

So I guess this is more of a notification to anyone wanting to run the auto uploader that it may not currently be working. Are there any alternatives we could look into in the meantime or even as a permanent solution/replacement?

@jmhobbs
Copy link
Owner

jmhobbs commented Dec 18, 2018

Yeah, looks like an emojipacks issue to me. I'm up for a warning in the README. Slack has no API for uploading emoji, only listing them, so there's not really a better option except for browser automation/emulation. I'm open to ideas though!

@Darth-Arminius
Copy link
Contributor Author

Darth-Arminius commented Dec 18, 2018

Ah, well that's a great shame. I'll have a little look into it and see what our alternatives could be. I agree that in the meantime a warning in the README would be a good idea.

@Darth-Arminius
Copy link
Contributor Author

Looking through, it seems most (if not all) batch slack emoji auto uploaders are broken as of Slack's latest update to the emoji uploading/managing UI. However, I did find a working chrome extension that allows for a slightly more manual way of batch uploading emojis. You drag and drop all the emoji files into this extension's UI and it will batch upload them all automatically. Extension can be found here: https://chrome.google.com/webstore/detail/neutral-face-emoji-tools/anchoacphlfbdomdlomnbbfhcmcdmjej

@jmhobbs
Copy link
Owner

jmhobbs commented Dec 21, 2018

Looks good to me, mind adding a quick PR with a warning and a link to the Chrome extension?

@Darth-Arminius
Copy link
Contributor Author

No problem, will do as soon as I can.

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

No branches or pull requests

2 participants