You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If req.ClientType is policies.UserType, then we might need to check user is present in domain or not.
I'm not sure about this , but i need to test this case.
Steps to replicate :
We need to add member to domain role
Add access to channel
Remove member from domain role
then test reading value of message via reader.
On removal of member from domain role and if member is not removed from other entities and removed only from domains, then even after removal of member will still have access to channels.
User removal PR will solve this case.
Indicate the importance of this feature to you.
Must-have
Anything else?
No response
The text was updated successfully, but these errors were encountered:
arvindh123
changed the title
Test User Access After Removal from Domain Role (Issue with Channel Access)"
Test User Access After Removal from Domain Role (Issue with Channel Access)
Mar 20, 2025
arvindh123
changed the title
Test User Access After Removal from Domain Role (Issue with Channel Access)
Test User Access to Channels After Removal from Domain Role
Mar 20, 2025
Is your feature request related to a problem? Please describe.
No
Describe the feature you are requesting, as well as the possible use case(s) for it.
As mentioned here #2765 (comment)
If
req.ClientType
ispolicies.UserType
, then we might need to check user is present in domain or not.I'm not sure about this , but i need to test this case.
Steps to replicate :
We need to add member to domain role
Add access to channel
Remove member from domain role
then test reading value of message via reader.
On removal of member from domain role and if member is not removed from other entities and removed only from domains, then even after removal of member will still have access to channels.
User removal PR will solve this case.
Indicate the importance of this feature to you.
Must-have
Anything else?
No response
The text was updated successfully, but these errors were encountered: