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

Content Palette Should Not Show Archived Content #24292

Closed
1 task
Tracked by #24317
damen-dotcms opened this issue Mar 6, 2023 · 3 comments · Fixed by #24389
Closed
1 task
Tracked by #24317

Content Palette Should Not Show Archived Content #24292

damen-dotcms opened this issue Mar 6, 2023 · 3 comments · Fixed by #24389
Assignees
Labels
LTS: Excluded Ticket that has been excluded from at least one LTS OKR : User Experience Owned by Jason QA : Passed Internal Release : 23.01.9 Included in LTS patch release 23.01.9 Release : 23.05 Team : Lunik

Comments

@damen-dotcms
Copy link
Contributor

damen-dotcms commented Mar 6, 2023

Screenshot_2023-03-06_23-05-12.png

Problem Statement

On dotcms.com, the content palette for content type "customers" is showing logos for old customers that have been archived in dotCMS.

@fmontes The content palette is showing archived contentlets, the user should not see these while editing the page because they can only add non-archived content to it pages.

Steps to Reproduce

  1. Go to content search and archive, some contents of a specific content types
  2. Create or edit a page to show the content palette
  3. In the content palette, filter by the content types of the content you archived
  4. ERROR: You should be seeing the archived contentlets

Acceptance Criteria

Tasks

Additional Information

I think the problem here is related to the query we do to the API to get those contentlets.


Reported by: Melissa Rojas Rodríguez (melissa.rojas@dotcms.com)

Source URL: https://authoring.dotcms.com/dotAdmin/#/edit-page/content?url=%2Findustries%2Fmedia-and-entertainment&language_id=1&host_id=8a7d5e23-da1e-420a-b4f0-471e7da8ea2d

Issue details: Open in Marker.io

Device typedesktop
BrowserChrome 110.0.0.0
Screen Size2134 x 1200
OSWindows 15.0.0
Viewport Size2134 x 1030
Zoom Level100%
Pixel Ratio@​1.8000000715255737x
@erickgonzalez erickgonzalez added the LTS: Reproduce Unknown if this issue exists in current LTS releases label Mar 7, 2023
@damen-dotcms damen-dotcms changed the title Content palette showing archived content Content Palette Should Not Show Archived Content Mar 7, 2023
@zJaaal zJaaal removed their assignment Mar 14, 2023
@fmontes
Copy link
Member

fmontes commented Mar 15, 2023

To the query we need to add +deleted:false so it doesn't return archived items.

@zJaaal zJaaal self-assigned this Mar 15, 2023
@zJaaal zJaaal linked a pull request Mar 15, 2023 that will close this issue
3 tasks
fmontes pushed a commit that referenced this issue Mar 20, 2023
* add query to ES to not retrieve archive content

* fix broken test
@rjvelazco
Copy link
Contributor

Passed Internal QA

Docker Image: [dotcms/dotcms:master_70229c6b]

internal-qa-issue-24292-content-palette-should-not-show-archived-content.mov

@josemejias11
Copy link

Approved QA - Tested on 23.05_03732ac4_SNAPSHOT // Docker // macOS 13.0 // FF v111.0

@erickgonzalez erickgonzalez added LTS : Next Ticket that will be added to LTS Next LTS Release and removed LTS: Reproduce Unknown if this issue exists in current LTS releases LTS : Next Ticket that will be added to LTS labels Nov 7, 2023
erickgonzalez added a commit that referenced this issue Nov 17, 2023
@erickgonzalez erickgonzalez added the Release : 23.01.9 Included in LTS patch release 23.01.9 label Nov 27, 2023
@erickgonzalez erickgonzalez added LTS: Excluded Ticket that has been excluded from at least one LTS and removed Next LTS Release labels Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LTS: Excluded Ticket that has been excluded from at least one LTS OKR : User Experience Owned by Jason QA : Passed Internal Release : 23.01.9 Included in LTS patch release 23.01.9 Release : 23.05 Team : Lunik
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants