Skip to content
This repository was archived by the owner on Jan 15, 2022. It is now read-only.

Fix/5.3.x/maint 23038 #546

Open
wants to merge 2 commits into
base: stable/5.3.x
Choose a base branch
from
Open

Fix/5.3.x/maint 23038 #546

wants to merge 2 commits into from

Conversation

mzorai
Copy link
Contributor

@mzorai mzorai commented Jun 12, 2020

No description provided.

* PLF-8674: Disabled users' display issues
* PLF-8674:Ignore disabled users from suggestion
* PLF-8674: update Unit tests
Co-authored-by: Ali HAMDI <[email protected]>

(cherry picked from commit 411c534)
When a user is disabled , there a missing indications in  :
- User profile page
- User popup
The fix will add the word disabled next to the username
@mzorai mzorai requested review from ahamdi and lbengayed June 12, 2020 12:47
Copy link
Member

@ahamdi ahamdi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, we need a dedicated PR for 5.3.

@mzorai
Copy link
Contributor Author

mzorai commented Jun 12, 2020

Unfortunately, we need a dedicated PR for 5.3.

Ah OK, I made some changes manually as it has conflicts, the build passed well but I did not yet made tests, so should I close this PR?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants