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

On Zulip, ROOMS_TO_JOIN should contain valid Zulip rooms #635

Closed
jayvdb opened this issue Oct 24, 2018 · 4 comments · Fixed by #639 or why-does-ie-still-exist/corobo#1
Closed

On Zulip, ROOMS_TO_JOIN should contain valid Zulip rooms #635

jayvdb opened this issue Oct 24, 2018 · 4 comments · Fixed by #639 or why-does-ie-still-exist/corobo#1

Comments

@jayvdb
Copy link
Member

jayvdb commented Oct 24, 2018

Following on from #634 , the rooms list needs to be updated on backend Zulip to contain only rooms which exist on the coala Zulip server , otherwise there are lots of errors on startup.

Note that 'maintainers' (same as gitter coala/coala/maintainers) and 'gci-mentors-2018' (no gitter equivalent) are two private rooms the bot also needs to join on Zulip.

@jayvdb
Copy link
Member Author

jayvdb commented Oct 24, 2018

@why-does-ie-still-exist
Copy link
Contributor

@jayvdb does the bot need to scrape zulip, use the zulip api, or is this just a static list of rooms pertinent to the bot?

@why-does-ie-still-exist
Copy link
Contributor

@jayvdb does the bot need to scrape zulip, use the zulip api, or is this just a static list of rooms pertinent to the bot?

Nevermind. I see what you mean(I think) I'll update the list. :-)

@jayvdb
Copy link
Member Author

jayvdb commented Oct 24, 2018

It can be a static list.

why-does-ie-still-exist added a commit to why-does-ie-still-exist/corobo that referenced this issue Oct 26, 2018
Added this code which if the `BACKEND` is run as `'Zulip'`
`ROOMS_TO_JOIN` is updated accordingly.
I noticed there's not much code for Zulip(a cursory
search shows no matches in the repo). So this
will probably make someone's life easier down
the line when they port the bot.

closes coala#635

Update config.py
why-does-ie-still-exist added a commit to why-does-ie-still-exist/corobo that referenced this issue Oct 26, 2018
Added this code which if the `BACKEND` is run as `'Zulip'`
`ROOMS_TO_JOIN` is updated accordingly.
I noticed there's not much code for Zulip(a cursory
search shows no matches in the repo). So this
will probably make someone's life easier down
the line when they port the bot.

closes coala#635
why-does-ie-still-exist added a commit to why-does-ie-still-exist/corobo that referenced this issue Oct 26, 2018
Added this code which if the `BACKEND` is run as `'Zulip'`
`ROOMS_TO_JOIN` is updated accordingly.
I noticed there's not much code for Zulip(a cursory
search shows no matches in the repo). So this
will probably make someone's life easier down
the line when they port the bot.

closes coala#635
why-does-ie-still-exist added a commit to why-does-ie-still-exist/corobo that referenced this issue Oct 26, 2018
Added this code which if the `BACKEND` is run as `'Zulip'`
`ROOMS_TO_JOIN` is updated accordingly.
I noticed there's not much code for Zulip(a cursory
search shows no matches in the repo). So this
will probably make someone's life easier down
the line when they port the bot.

closes coala#635
jayvdb pushed a commit to why-does-ie-still-exist/corobo that referenced this issue Oct 26, 2018
Added this code which if the `BACKEND` is run as `'Zulip'`
`ROOMS_TO_JOIN` is updated accordingly.

Closes coala#635
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment