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

Deactivated objects can still be accessed using shortcuts & cmd+k #6776

Open
BOHEUS opened this issue Aug 28, 2024 · 3 comments · May be fixed by #7016
Open

Deactivated objects can still be accessed using shortcuts & cmd+k #6776

BOHEUS opened this issue Aug 28, 2024 · 3 comments · May be fixed by #7016
Labels
good first issue Good for newcomers scope: front Issues that are affecting the frontend side only size: short type: bug Something isn't working

Comments

@BOHEUS
Copy link
Contributor

BOHEUS commented Aug 28, 2024

Scenario:

  1. Log in
  2. Go to Settings > Data Model
  3. Click on any active object, e.g. Companies, then on three dots near "Standard" label and click "Deactivate"
  4. Go back to main page and use corresponding shortcut to access deactivated objects

Actual: Deactivated objects can still be accessed using shortcuts
Screencast from 2024-08-28 20-43-15.webm

Expected: Deactivated objects shouldn't be accessed using shortcuts until they're re-activated by the user.

@BOHEUS BOHEUS added the type: bug Something isn't working label Aug 28, 2024
@Bonapara Bonapara changed the title Deactivated objects can still be accessed using shortcuts Deactivated objects can still be accessed using shortcuts & cmd+k Aug 29, 2024
@Bonapara Bonapara added scope: front Issues that are affecting the frontend side only size: short labels Aug 29, 2024
@Bonapara
Copy link
Member

This should be tackle when we will work on the cmd+k

@charlesBochet
Copy link
Member

Should still be easy to change in the current code base, tagging it as good first issue

@charlesBochet charlesBochet added the good first issue Good for newcomers label Sep 1, 2024
@sid0-0 sid0-0 linked a pull request Sep 13, 2024 that will close this issue
@FelixMalfait
Copy link
Member

@sid0-0 could you please comment on the issue to let me assign it to you? Thanks a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers scope: front Issues that are affecting the frontend side only size: short type: bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants