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

Edit EMA: Respect User Permissions #27135

Closed
3 tasks
Tracked by #26644
fmontes opened this issue Jan 2, 2024 · 0 comments · Fixed by #27139
Closed
3 tasks
Tracked by #26644

Edit EMA: Respect User Permissions #27135

fmontes opened this issue Jan 2, 2024 · 0 comments · Fixed by #27139
Assignees
@KevinDavilaDotCMS
Labels
OKR : Core Features Owned by Will Priority : 3 Average QA : Not Needed Team : Lunik Type : New Functionality

Comments

@fmontes
Copy link
Member

fmontes commented Jan 2, 2024 •
edited
Loading

Parent

#26644

User Story

As a dotCMS user, I want to view and edit a page while respecting my user permissions, so I can access the page and its content securely.

Acceptance Criteria
Preview Give feedback

Proposed Objective

Core Feature

Proposed Priority

Priority 3 - Average

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.:

In the page API response, we have page.canEdit we should use that.

Assumptions & Initiation Needs

N/A

Quality Assurance Notes & Workarounds

N/A

Sub-Tasks & Estimates

  • Design a way to convey to the user that they have no permission to edit the page.
  • Create an error handler service for when the user has no permission to view the page.
  • Hide the palette when the user has permission
@fmontes fmontes mentioned this issue Jan 2, 2024
Closed
@KevinDavilaDotCMS KevinDavilaDotCMS self-assigned this Jan 2, 2024
@KevinDavilaDotCMS KevinDavilaDotCMS linked a pull request Jan 3, 2024 that will close this issue
feat(edit-ema): Show messages for user permissions #27135 #27139
Merged
@zJaaal zJaaal moved this from New to In Progress in dotCMS - Product Planning Jan 3, 2024
@KevinDavilaDotCMS KevinDavilaDotCMS moved this from In Progress to In Review in dotCMS - Product Planning Jan 3, 2024
@fmontes fmontes moved this from In Review to Internal QA in dotCMS - Product Planning Jan 10, 2024
github-merge-queue bot pushed a commit that referenced this issue Jan 10, 2024
@KevinDavilaDotCMS
feat(edit-ema): Show messages for user permissions #27135 (#27139)
12e9c11 
* Added check read and edit permissions

* Replaced hardcoded css value to variable

* Change cursor type on palette

* Added validation to Http fail when dont have permissions

* Added disabled prop on disabled options in Edit EMA navigation bar

* Add mechanism to react to Http fails

* Added NotFout component on Edit EMA

* Added AccessDenied component

* Moved NotLicense to ui. Added InfoPage component. Added new pages based on permissions in Edit EMA

* Update comment about notLicensed lazy component

* Added dot-not-license-page to fix reference to lib in routing

* Add lint

* Fix reference of component

* Fix reference of component

* Added permissions and license.

* Create re-usable resolver to enterprise portlets

* Working on PR suggestions

* Fix on UI lib test

* Fixed color on p

* Added new UI design when the user dont have edit permission on Edit EMA
@github-project-automation github-project-automation bot moved this from Internal QA to Done in dotCMS - Product Planning Jan 10, 2024
@fmontes fmontes moved this from Done to Internal QA in dotCMS - Product Planning Jan 12, 2024
@fmontes fmontes moved this from Internal QA to Done in dotCMS - Product Planning Jan 16, 2024
@KevinDavilaDotCMS KevinDavilaDotCMS mentioned this issue Oct 25, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KevinDavilaDotCMS KevinDavilaDotCMS

Labels
OKR : Core Features Owned by Will Priority : 3 Average QA : Not Needed Team : Lunik Type : New Functionality
Projects
dotCMS - Product Planning
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(edit-ema): Show messages for user permissions #27135 dotCMS/core
2 participants
@fmontes @KevinDavilaDotCMS