Arthri
Décembre 17, 2022, 3:30
1
TL0 a pu envoyer un message à un groupe dont le paramètre « Qui peut envoyer un message à ce groupe ? » était défini sur « Tout le monde » grâce à une fonctionnalité implémentée ici
Is there any way to allow TL0 members to start messages to a specific discourse group, but not to other regular members or non staff members? I am currently not finding a way to make this work. When TL0 members write to our helpdeskteam group for help, they get an off putting and confusing error message - see below.
[18%20PM]
The TL0 user is writing to a group with “who can message this member” set to “everyone”. If I make the user TL1 the message is saved with no error.
Everyone in the group…
Cependant, ce n’est plus possible en raison d’une mise à jour au cours du dernier mois (notre version du forum est la 2.9.0.beta14). Je pense que cette PR a causé la régression
main ← issue/can-send-pm-check-not-reliant-on-system-user
opened 05:41AM - 01 Nov 22 UTC
Since the system user is a regular user, it can have its `allow_private_messages… ` user option turned off, which with our current `can_send_private_message?(Discourse.system_user)` check inside the CurrentUserSerializer, will prevent any user from sending messages in the UI if the system user is not accepting PMs.
This commit adds a new `can_send_private_messages?` method to the Guardian, which can be used in serializers and not depend on the system user. When the user actually sends a message we still rely on the old `can_send_private_message?(target)` call to see if they are allowed to send the message to the target. The new method is just to say they can "generally" send private messages.
2 « J'aime »
martin
Décembre 20, 2022, 3:11
3
Merci pour le rapport @Arthri , c’est corrigé ici :
main ← issue/fix-messageable-everyone-regression-tl0
opened 01:14AM - 20 Dec 22 UTC
The commits https://github.com/discourse/discourse/commit/e62e93f83a77adfa80b38f… bfecf82bbee14e12fe and
https://github.com/discourse/discourse/commit/d6bd4ad7ee1d3bf43ba0a2bb1948f8f306b6177a caused a regression
to the behaviour added for
https://meta.discourse.org/t/allow-tl0-to-write-messages-to-staff-group-not-to-other-members-or-non-staff/124335,
which allowed a user to message a group with the messageable_level
set to Everyone even if they were TL0 (or otherwise did not reach
the appropriate trust level). This commit fixes the issue and adjusts
the spec to reflect the real scenario.
c.f. https://meta.discourse.org/t/tl0-cant-message-groups-with-messageable-level-everyone-recession/249205
2 « J'aime »
JammyDodger
Décembre 24, 2022, 8:00
4
Ce sujet a été automatiquement fermé après 25 heures. Les nouvelles réponses ne sont plus autorisées.