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

IBX-8254: Added check if relation is accessible #1260

Merged

Conversation

mateuszdebinski
Copy link
Contributor

🎫 Issue IBX-8254

@mateuszdebinski mateuszdebinski force-pushed the IBX-8254_error_relation_list_without_permission_to_content branch from 49f1684 to c8c3f16 Compare May 7, 2024 14:44
Copy link

sonarcloud bot commented May 7, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@barbaragr
Copy link

In relations tab there is a message displayed. I am not sure if we should reveal ID of content item that user doesn't have permission to read. Maybe member of UX team should look at that?
Screenshot 2024-05-09 at 14 20 57

@dew326 dew326 merged commit a5e7552 into 4.6 May 10, 2024
22 checks passed
@dew326 dew326 deleted the IBX-8254_error_relation_list_without_permission_to_content branch May 10, 2024 06:02
GrabowskiM pushed a commit that referenced this pull request May 27, 2024
Co-authored-by: Mateusz Dębiński <mateusz.debinski@ibexa.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working QA approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants