-
-
Notifications
You must be signed in to change notification settings - Fork 640
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
refactor updateExistingStream: Reduce api calls on changing stream details. #3222
Comments
jainkuniya
added a commit
to jainkuniya/zulip-mobile
that referenced
this issue
Dec 18, 2018
Before inidividual api calls were made for each stream property which is changed. But on server, this api is capable of handeling multiple propery at a time. So just call api once with all the properties which are changed. Fixes: zulip#3222
jainkuniya
added a commit
to jainkuniya/zulip-mobile
that referenced
this issue
Jan 1, 2019
Before inidividual api calls were made for each stream property which is changed. But on server, this api is capable of handeling multiple propery at a time. So just call api once with all the properties which are changed. Fixes: zulip#3222
jainkuniya
added a commit
to jainkuniya/zulip-mobile
that referenced
this issue
Jan 19, 2019
Before inidividual api calls were made for each stream property which is changed. But on server, this api is capable of handeling multiple propery at a time. So just call api once with all the properties which are changed. Fixes: zulip#3222
jainkuniya
added a commit
to jainkuniya/zulip-mobile
that referenced
this issue
Jan 19, 2019
Before inidividual api calls were made for each stream property which is changed. But on server, this api is capable of handeling multiple propery at a time. So just call api once with all the properties which are changed. Fixes: zulip#3222
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently if user changes stream name and description, then we are making two API calls. On the other hand
streams
API endpoint is generic and can handle multiple properties at the same time. So redactorupdateExistingStream
method, so that in such cases only one request is send to server.https://chat.zulip.org/#narrow/stream/3-backend/topic/api.20.2Fstreams.2F.24.7Bid.7D
@zulipbot claim
The text was updated successfully, but these errors were encountered: