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.

Sign up for GitHub

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

Jump to bottom

The user should not be able to update activity without permission. #709

Closed
Sanjaybhattwebkul opened this issue Nov 16, 2021 · 1 comment
Closed

The user should not be able to update activity without permission. #709

Sanjaybhattwebkul opened this issue Nov 16, 2021 · 1 comment
Assignees
@jitendra-webkul
Labels
Bug Fixed Bug

Comments

@Sanjaybhattwebkul
Copy link
Contributor

Sanjaybhattwebkul commented Nov 16, 2021 •
edited
Loading

CRM Version: V1.2.1

Steps To reproduce--

1. Log in as Admin/User.
2. Create a new user do not give permission to edit activity.
3. Log in as a new user.
4. Click on the Activity tab
5. Try to update activity by a radio button or mass action.

Actual result: The user is able to update activity by radio button or mass action without permission. Please find the below screenshot.

Expected result: The user should not be able to update activity by using the radio button or mass action if the user has no permission for the same.

Act-1

Act-2
@Sanjaybhattwebkul Sanjaybhattwebkul changed the title The user should be able to update activity from the activity table. The user should not be able to update activity without permission from the activity table. Nov 16, 2021
@Sanjaybhattwebkul Sanjaybhattwebkul changed the title The user should not be able to update activity without permission from the activity table. The user should not be able to update activity without permission. Nov 16, 2021
jitendra-webkul added a commit to jitendra-webkul/bagisto-crm that referenced this issue Nov 16, 2021
@jitendra-webkul
Issue krayin#709 fixed
54bf017
@jitendra-webkul jitendra-webkul mentioned this issue Nov 16, 2021
Merged
jitendra-webkul added a commit that referenced this issue Nov 16, 2021
@jitendra-webkul
Merge pull request #711 from jitendra-webkul/master
1e23836 
Issue #709 fixed
@Sanjaybhattwebkul
Copy link
Contributor Author

@jitendra-webkul
Thanks, now this issue is fixed. so I am closing this issue.
ANd I have attached a screenshot for future reference

fixed-act
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jitendra-webkul jitendra-webkul

Labels
Bug Fixed Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jitendra-webkul @Sanjaybhattwebkul