-
Notifications
You must be signed in to change notification settings - Fork 1
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
Put a timed delay in between each welcome message #1
Labels
Comments
@sethetter I will try to solve this. jQuery allowed? |
@bikrambhattacharya this is running on the server side, so jQuery won't be relevant here. Should be achievable with Promises. |
@sethetter okay I will try then not sure . |
@bikrambhattacharya if you have trouble feel free to post here! |
@sethetter I am working on this. |
Awesome! Looking forward to seeing it :)
…On Jul 12, 2017 5:23 AM, "Bikram Bhattacharya" ***@***.***> wrote:
@sethetter <https://github.com/sethetter> I am working on this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAoAjNGRKsvPDtNP93SCVSCCWldoeJgGks5sNJ6jgaJpZM4OR4sa>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In
/scripts/onboarding.js
, we currently send all the messages at once, which can be a lot. It would be a bit of a better experience for the new comer if they received each message with at least 10-15 second (maybe longer?) pause in between.This is possible with some
setTimeout
and promise chaining! Should be a fun little challenge. Who's up for it?!To do this and not have to test with the live Slack, just update the plugin to reply to something instead of using
robot.enter()
. After the delay is working, update the code in therobot.enter()
handler.The text was updated successfully, but these errors were encountered: