coala / devops

MIT License
9 stars 39 forks source link

make cobot join private chat with all coala members #23

Closed coala-bot closed 7 years ago

coala-bot commented 7 years ago

@yukiisbored here is the script that you'd have to run. It will need:

You can just join one user by using it like this python3 join_coalaians.py --user <username>

I'd prefer that you make SEND_MESSAGE True which will send a message to all the people with whom cobot will join rooms that cobot is accessible on private chats from now on... NOTE 1: ^^ If you run the script again, then it'll send the message again to that person, so if possible allow script to run completely, on all members.

NOTE 2: Please have a look at the logs once and join rooms with users that were not able to join with python3 join_coalaians.py --user <username>

NOTE 3: Please restart cobot after running this script

Thanks! cobot is accessible on Private chats as well, now! :tada:

Opened via gitter by @meetmangukiya

meetmangukiya commented 7 years ago

We will probably want to run this as a cron in future, but for now, please run it once manually, so that cobot is atleast functional for current members :)

Also by then I'll clean up the script, to check if cobot has already joined the room and in that case not send the message again :+1:

jayvdb commented 7 years ago

why not do the same from the 'cobot invite' command?

meetmangukiya commented 7 years ago

why not do the same from the 'cobot invite' command?

Yeah, we should do that.. But anyways we'd have to run the script once for existing members once and also once before merging the above feature to join with the difference of the members

@yukiisbored no need to setup the cron for now, just run it once :+1:

yukiisbored commented 7 years ago

Yeah, I think it's better to make it a part of cobot invite rather than a heavy cron job...

meetmangukiya commented 7 years ago

@yukiisbored bump ?

yukiisbored commented 7 years ago

Executed. Exit code 0

meetmangukiya commented 7 years ago

Thanks! Btw @yukiisbored did or did you not set the SEND_MESSAGE variable? And did you look at the logs? Did they look good? I didn't use/manage exit codes in the script

yukiisbored commented 7 years ago

@meetmangukiya No errors, I set SEND_MESSAGE to False

meetmangukiya commented 7 years ago

Ok I'll make an announcement on coala channel later today then, thanks!

meetmangukiya commented 7 years ago

Seems like this isn't really working for all the members. Try running again with Send_Message as True. Also, we should probably log status codes as well, it'll make it easier to debug if it still doesn't work.

(Re-opening)

yukiisbored commented 7 years ago

Closed, due to future replacement to corobo