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

LPS-144732 Fix Info Panel when the user has been deleted #4550

Closed
wants to merge 3 commits into from
Closed

LPS-144732 Fix Info Panel when the user has been deleted #4550

wants to merge 3 commits into from

Conversation

cgoncas
Copy link
Contributor

@cgoncas cgoncas commented Jan 20, 2022

Motivation*

Content Dashboard shows error when displaying details of article whose authors has been deleted

Link to story or ticket

Proposed Solution*

Use the content dashboard user id instead the info framework authorPortraitURL

Steps to Verify*:

  1. Go to Control Panel→Instance Settings→User Authentication
  2. Uncheck Require password for email or screen name updates? → Save
  3. Go to Control Panel → Users and Organizations and click +
  4. Set the user's screen name to test-editor and fill the remaining required fields with random data
  5. Go to Memberships tab and assign the user to the current site → Save
  6. Go to Roles tab and assign the user a Site Administrator role of the current site → Save
  7. Go to Password tab and set a random password that will be later used to log in
  8. Open Incognito mode of your browser
  9. Log in as the user test-editor
  10. Enter a new password if asked
  11. Go to Content & Data → Web Content, click + and select Basic Web Context
  12. Fill the article with completely random data and publish
  13. Log out as user test-editor and close the incognito mode tab
  14. Go back to main browser (being logged as an administrator)
  15. Go to Control Panel → Users and Organizations
  16. Find the test-editor on the list, click on the context menu (3 dots) and click Deactivate
  17. Click on Filter and Order and select the Inactive filter
  18. Find the test-editor on the list, click on the context menu (3 dots) and click Delete
  19. Go to Applications → CONTENT → Content Dashboard
  20. Find the previously created article and click on the icon ℹ️

@cgoncas cgoncas closed this Jan 20, 2022
liferay-continuous-integration-hu pushed a commit to evibudai/liferay-portal that referenced this pull request Jan 26, 2022
@brianchandotcom
LPS-144181 Update Clay dependencies
7fd95b6 
 ## [3.44.2](liferay/clay@v3.44.1...v3.44.2) (2022-01-12)

 ### Bug Fixes

 -   **@clayui/css:** Cadmin Nav adds `background-color` to `.active` pseudo element ([865b037](liferay/clay@865b037)), closes [liferay#4562](liferay/clay#4562)
 -   **@clayui/css:** Mixins `clay-button-variant` removes unnecessary `setter()`'s ([3709d7f](liferay/clay@3709d7f)), closes [liferay#4550](liferay/clay#4550)

 ## [3.44.1](liferay/clay@v3.44.0...v3.44.1) (2022-01-11)

 ### Bug Fixes

 -   **@clayui/core:** remove the context property from DndProvider ([4faf0ea](liferay/clay@4faf0ea))
 -   **@clayui/css:** Mixins `clay-card-variant` allow styling before and after psuedo elements ([69f1d38](liferay/clay@69f1d38)), closes [liferay#4554](liferay/clay#4554)

 # [3.44.0](liferay/clay@v3.43.1...v3.44.0) (2022-01-10)

 ### Bug Fixes

 -   **@clayui/css:** Cadmin Input Groups `input-group-sm` missing mixin declaration ([d7027be](liferay/clay@d7027be)), closes [liferay#4537](liferay/clay#4537)
 -   **@clayui/css:** Cards `form-check-card` remove duplicate hover state style ([17ea640](liferay/clay@17ea640))
 -   **@clayui/css:** Mixins Cards check if parameter is map to avoid must be a map error ([2174587](liferay/clay@2174587))
 -   **@clayui/css:** Mixins Custom Forms remove `setter()`, no longer needed ([4cb30ce](liferay/clay@4cb30ce))

 ### Features

 -   **@clayui/css:** Cadmin Nav adds `nav-divider` and `nav-divider-end` ([66ba6ce](liferay/clay@66ba6ce))
 -   **@clayui/css:** Cards `form-check-card` checkbox/radio input should be hidden unless position utilities are used ([3120797](liferay/clay@3120797)), closes [liferay#4544](liferay/clay#4544)
 -   **@clayui/css:** Cards `form-check-card` convert to Clay mixin pattern ([5b4424b](liferay/clay@5b4424b))
 -   **@clayui/css:** Mixin `clay-card-variant` make `form-check-card` and `form-check-input` customizable ([c6a3a6f](liferay/clay@c6a3a6f))
 -   **@clayui/css:** Mixins `clay-custom-control-input-variant` add option to customize card ([87682d2](liferay/clay@87682d2))
 -   **@clayui/css:** Mixins `clay-navbar-variant` adds option to customize `nav-divider` and `nav-divider-end` ([e58c23e](liferay/clay@e58c23e))
 -   **@clayui/css:** Mixins Card adds `clay-form-check-card-variant` ([a834502](liferay/clay@a834502))
 -   **@clayui/css:** Nav adds `nav-divider` and `nav-divider-end` ([600a379](liferay/clay@600a379))
 -   **@clayui/css:** SVG Icons adds date-time ([4bf8d4c](liferay/clay@4bf8d4c))
 -   **@clayui/popover:** add a closeOnClickOutside prop ([bd722a9](liferay/clay@bd722a9)), closes [liferay#4536](liferay/clay#4536)

 ## [3.43.1](liferay/clay@v3.43.0...v3.43.1) (2022-01-04)

-   **@clayui/core:** avoid async/await in TreeViewItem ([78f8585](liferay/clay@78f8585)), closes [liferay#4529](liferay/clay#4529) [liferay#4531](liferay/clay#4531)
-   **@clayui/core:** don't export TreeView as UNSAFE_TreeView ([fc54a6e](liferay/clay@fc54a6e)), closes [liferay#4528](liferay/clay#4528) [liferay#4531](liferay/clay#4531) [liferay#4532](liferay/clay#4532)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cgoncas