-
Notifications
You must be signed in to change notification settings - Fork 162
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
Hide compose box based on stream posting policy #674
Labels
a-compose
Compose box, autocomplete, attaching files/images
Milestone
Comments
This was referenced Aug 11, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When this Zulip feature is in effect:
https://zulip.com/help/stream-sending-policy
it may mean that the self-user wouldn't be able to send to a given stream. When that's the case, we should replace the compose box with a banner explaining this is the case.
In the API, this is
ZulipStream.streamPostPolicy
.In zulip-mobile we have a version of this, but it only detects "announcement-only" streams, i.e. where the policy is
StreamPostPolicy.administrators
. (Because long ago that was the only version that existed.) Just recently I happened to see a user get affected by this on a stream where the policy wasStreamPostPolicy.moderators
and they weren't a moderator — the symptom (on zulip-mobile) was that the app tries to send and then the message just sits there with an outbox spinner forever. So that underlines that this feature is a meaningful one which we should be sure to implement (and do so completely).The text was updated successfully, but these errors were encountered: