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

[tests-only][full-ci]Added display role text change before assertion #8934

Merged
merged 1 commit into from
Apr 27, 2023

Conversation

SagarGi
Copy link
Member

@SagarGi SagarGi commented Apr 27, 2023

Description

Beacuse of the change of this PR #8876. For public link the display text after the selection of different role is now different. So this PR maps different role of public link to different role in displayed in the UI.

Related Issue:

#8930

@SagarGi SagarGi self-assigned this Apr 27, 2023
@SagarGi SagarGi requested review from SwikritiT and saw-jan April 27, 2023 07:00
@ownclouders
Copy link
Contributor

@SagarGi SagarGi force-pushed the fix_changing_role_webUI branch from 8d289ce to 3645b88 Compare April 27, 2023 07:24
@SagarGi SagarGi force-pushed the fix_changing_role_webUI branch from 3645b88 to c117e52 Compare April 27, 2023 08:25
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@SagarGi SagarGi changed the title [tests-only][full-ci]Added display role text change before assertion [tests-only]Added display role text change before assertion Apr 27, 2023
@SagarGi SagarGi changed the title [tests-only]Added display role text change before assertion [tests-only][full-ci]Added display role text change before assertion Apr 27, 2023
@SagarGi SagarGi marked this pull request as ready for review April 27, 2023 09:15
@SagarGi SagarGi requested a review from ScharfViktor April 27, 2023 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants