Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Add comment about /json/subscriptions/exists endpoint. 

We should remove this endpoint.  For now I'm just adding
the comment to the code explaining why we should do that.
This commit is contained in:
Steve Howell 2016-10-17 14:49:03 -07:00 committed by Tim Abbott
parent 906a4e3b26
commit ff6404cb3c
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
zproject/legacy_urls.py
View File

@ -13,7 +13,14 @@ legacy_urls = [
url(r'^json/notify_settings/change$', 'zerver.views.user_settings.json_change_notify_settings'), url(r'^json/notify_settings/change$', 'zerver.views.user_settings.json_change_notify_settings'),
url(r'^json/ui_settings/change$', 'zerver.views.user_settings.json_change_ui_settings'), url(r'^json/ui_settings/change$', 'zerver.views.user_settings.json_change_ui_settings'),
url(r'^json/subscriptions/remove$', 'zerver.views.streams.json_remove_subscriptions'), url(r'^json/subscriptions/remove$', 'zerver.views.streams.json_remove_subscriptions'),
# We should remove this endpoint and all code related to it.
# It returns a 404 if the stream doesn't exist, which is confusing
# for devs, and I don't think we need to go to the server
# any more to find out about subscriptions, since they are already
# pushed to us via the event system.
url(r'^json/subscriptions/exists$', 'zerver.views.streams.json_stream_exists'), url(r'^json/subscriptions/exists$', 'zerver.views.streams.json_stream_exists'),
url(r'^json/subscriptions/property$', 'zerver.views.streams.json_subscription_property'), url(r'^json/subscriptions/property$', 'zerver.views.streams.json_subscription_property'),
url(r'^json/get_subscribers$', 'zerver.views.streams.json_get_subscribers'), url(r'^json/get_subscribers$', 'zerver.views.streams.json_get_subscribers'),
url(r'^json/fetch_api_key$', 'zerver.views.auth.json_fetch_api_key'), url(r'^json/fetch_api_key$', 'zerver.views.auth.json_fetch_api_key'),