Skip to content

ComposeBox: Don't require caughtUp to send in stream narrow.#4772

Closed
WesleyAC wants to merge 1 commit intozulip:masterfrom
WesleyAC:composebox-no-stream-caughtup
Closed

ComposeBox: Don't require caughtUp to send in stream narrow.#4772
WesleyAC wants to merge 1 commit intozulip:masterfrom
WesleyAC:composebox-no-stream-caughtup

Conversation

@WesleyAC
Copy link
Copy Markdown
Contributor

@WesleyAC WesleyAC commented Jun 1, 2021

See CZO 1 for some background on this. There are many reasons why the
destination narrow might not be the caughtUp narrow in the stream view,
and it's worse to prevent the user from sending with a cryptic error
message than it is to allow them to send to a narrow they aren't fully
caught up in.

See CZO [1] for some background on this. There are many reasons why the
destination narrow might not be the caughtUp narrow in the stream view,
and it's worse to prevent the user from sending with a cryptic error
message than it is to allow them to send to a narrow they aren't fully
caught up in.

[1]: https://chat.zulip.org/#narrow/stream/48-mobile/topic/Failed.20to.20send.20on.20Android
@WesleyAC WesleyAC requested review from chrisbobbe and gnprice June 1, 2021 21:09
@WesleyAC WesleyAC marked this pull request as draft June 2, 2021 05:21
@WesleyAC
Copy link
Copy Markdown
Contributor Author

WesleyAC commented Jun 2, 2021

Superseded by #4773.

@WesleyAC WesleyAC closed this Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant