-
Notifications
You must be signed in to change notification settings - Fork 238
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
Use max_message_length instead of hardcoded limit of 10000 #1237
Comments
Hey, I am working on it. I have successfully fetched the max_message_length from the server and added it in the models. I am now working on integrating it with the compose_box widget. |
Update : I did the integration with the sendbutton widget error checker function successfully but somehow i cant raise a PR now. Whenever i try to push the code to origin, it is giving me 403 error. |
Hey @Spyyy004, what is the url for the |
remote: Permission to zulip/zulip-flutter.git denied to Spyyy004. |
can you run |
Output of the command : https://github.com/zulip/zulip-flutter.git I am unable to raise a PR because I am unable to push the code to origin on my branch. |
Oh, that's where the problem is: You first need to fork zulip-flutter and then clone your fork. You can see this short video to get a better understanding of how to do it. |
Oh so i need to work on a forked repository. Got it. Thanks for the help |
Here is the PR : #1267 |
Awesome! |
We currently use a constant
kMaxMessageLengthCodePoints
. We should instead usemax_message_length
from the server; search formax_message_length
at https://zulip.com/api/register-queueRelated issues
The text was updated successfully, but these errors were encountered: