Skip to content
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

fix: Internal user marked as external in people and space members - EXO-71462 - Meeds-io/meeds#2572 #4175

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

Jihed525
Copy link
Contributor

@Jihed525 Jihed525 commented Nov 8, 2024

Before this change, the user is marked as external on his card or people page and space members after remove the user from /platform/externals. To resolve this problem, change the display condition value of this component from boolean to String. After this change, the user is marked as external on his hard for people page and space members.

(cherry picked from commit 98d066d)

…es - EXO-71462 - Meeds-io/meeds#2572. (#4172)

Before this change, when create a user and add him in
/platform/externals group and remove it from /platform/users group then
remove the user from /platform/externals and I put it in /platform/users
and go to people page/ spacex member app where the user is member, the
user is marked as external on his card. To resolve this problem, change
the display condition of this component from boolean to String. After
this change, The user is marked as external on his hard for people page
and space members.

(cherry picked from commit 98d066d)
@Jihed525 Jihed525 self-assigned this Nov 8, 2024
@Jihed525 Jihed525 enabled auto-merge (squash) November 8, 2024 14:00
@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Nov 8, 2024
Copy link

sonarcloud bot commented Nov 8, 2024

@Jihed525 Jihed525 merged commit 7f8814c into develop Nov 8, 2024
4 checks passed
@Jihed525 Jihed525 deleted the backportdev-71462 branch November 8, 2024 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
partialCIBuild Perform Partial CI Build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants