Tim Abbott
Tim Abbott
Sure, this works, merged, thanks @Lalit3716!
For `WEB_PUBLIC_STREAMS_ENABLED`, I think we should just post it in the stream as an API change to manage; there's plenty more things like it, and some may not be priorities....
Merged, thanks @prakhar1144!
If you can, it'd be nice if you can clean up the commits to follow the Zulip commit guidelines: https://zulip.readthedocs.io/en/latest/contributing/version-control.html#commit-messages We're less strict about them with this project (which we...
@cmalek just a bump on cleaning up the commit history.
This is a bit late of a reply, but @dill0wn or anyone else who comes across this, feel free to submit a new PR that cleans this up.
I haven't thought about this (though maybe the original maintainer did -- we have mostly done bug fixes and Python version support updates since adopting the project). I would try...
Makes sense to me; posted a quick comment on documentation.
Possibly we should also add a brief section at the bottom of the README mentioning this behavior, maybe with a title like "Usage without ldap installed".
Hi @schtibe! The Zulip project just adopted this package which had been idle for a long time, so if you have time to clean up and rebase this PR, we...