Dashboards are accessible even they are restricted using Desk

[chandradx]

I have restricted the Dashboards to particular user.
User is not able to see the other dashboards in the desk.

But when someone shared the link of the Restricted dashboard, he is able to access them.
As per the restrictions, it should work. But its not working.

[nextchamp-saqib]

I couldn't replicate the issue.

1. I created a user permission record to restrict this user to see only one dashboard
   

2. Logged in with that user and checked the dashboard list (shows only the allowed dashboard)
   

3. Tried to open another dashboard using the URL, throws permission error
   

[chandradx]

Which version you are using?

…

On Wed, 20 Nov, 2024, 12:32 pm Saqib Ansari, ***@***.***> wrote: I couldn't replicate the issue. 1. I created a user permission record to restrict this user to see only one dashboard ***@***.*** (view on web) <https://github.com/user-attachments/assets/3019274e-2a5f-4647-b38c-d9f97e292388> 2. Logged in with that user and checked the dashboard list (shows only the allowed dashboard) ***@***.*** (view on web) <https://github.com/user-attachments/assets/84e73ecc-011e-49ff-a5a0-8c01ea4fd385> 3. Tried to open another dashboard using the URL, throws permission error ***@***.*** (view on web) <https://github.com/user-attachments/assets/93b2b892-147f-4599-8433-6a9775263e29> — Reply to this email directly, view it on GitHub <#359 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BHX3HQGZL5X2WQXPI2KXV5L2BQXWVAVCNFSM6AAAAABSBV5B2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBXGY3DONRRHE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[nextchamp-saqib]

This was tried on main branch

[chandradx]

Frappe Insights: v1.1.3 (develop) This is the version I am using

…

On Wed, 20 Nov, 2024, 12:39 pm Saqib Ansari, ***@***.***> wrote: This was tried on main branch — Reply to this email directly, view it on GitHub <#359 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BHX3HQFSHCK3YPIGB2XOZHT2BQYRDAVCNFSM6AAAAABSBV5B2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBXGY3TQMJUGE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[nextchamp-saqib]

Please switch to the main branch. As the one you are using is very outdated. Check if the issue persists on the main branch.

[chandradx]

Ok, I will check.

…

On Wed, 20 Nov, 2024, 12:47 pm Saqib Ansari, ***@***.***> wrote: Please switch to the main branch. As the one you are using is very outdated. Check if the issue persists on the main branch. — Reply to this email directly, view it on GitHub <#359 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BHX3HQEE4AM353KUYPKAHZL2BQZQBAVCNFSM6AAAAABSBV5B2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBXGY4TEOBUGI> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[nextchamp-saqib]

Please reopen the issue once you are able to replicate it

[chandradx]

Please reopen the issue once you are able to replicate it

I am still able to see the same issue in the main branch as well

[nextchamp-saqib]

Can you share the steps that you have taken to replicate the issue? Or a sample replication on any frappe cloud site would help

[chandradx]

I will share the recording of the issue which I am getting

[chandradx]

Hello Saqib, I recreated the scenario and shared you recording of it on your Telegram. Please check and let me know what can be done to solve this issue.

…

On Mon, 9 Dec 2024 at 18:28, Saqib Ansari ***@***.***> wrote: Can you share the steps that you have taken to replicate the issue? Or a sample replication on any frappe cloud site would help — Reply to this email directly, view it on GitHub <#359 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BHX3HQCBK5QA6PQOCEOZHKL2EWHW7AVCNFSM6AAAAABSBV5B2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRXHA2TQMBUGA> . You are receiving this because you authored the thread.Message ID: ***@***.***>