Follow-up from "Allow groups to require licensed seats for Duo Chat"
The following discussion from !150072 (merged) should be addressed:
-
@lulalala started a discussion: (+2 comments) issue: I can't quite connect the comment with the expected result. If you can could you explain more explicitly in the comment?
I also don't quite understand why L602 is allowed.
The following discussion from !150072 (merged) should be addressed:
-
@brytannia started a discussion: question: do we need the first condition? my understanding that
Feature.enabled?(:duo_chat_requires_licensed_seat)
will always returnfalse
as we never enable it globally.
From @jarka comment:
I would like to remove the feature flag check from the policy file, it can be in endpoints or in a service. For more context, please check our MR for new permission conventions