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

Folder values don't Push Published #26486

Closed
EdgarPsda opened this issue Oct 20, 2023 · 3 comments
Closed

Folder values don't Push Published #26486

EdgarPsda opened this issue Oct 20, 2023 · 3 comments
Assignees
Labels
LTS: Excluded Ticket that has been excluded from at least one LTS OKR : Customer Support Owned by Scott Priority : 2 High Release : 22.03.12 Included in LTS patch release 22.03.12 Team : Bug Fixers Type : Defect

Comments

@EdgarPsda
Copy link

EdgarPsda commented Oct 20, 2023

Parent Issue

No response

Problem Statement

When folder values are updated, then Push Published, those changes are not visible on the receiver environment.

Steps to Reproduce

1.- Modify the title or sort order of any folder.
2.- Push Publish it.
3.- A success message on the PP will appear.
4.- The title or sort order is/are not modified on the receiver environment.
Screenshot 2023-10-20 at 11 42 44 AM
Screenshot 2023-10-20 at 11 43 04 AM
Screenshot 2023-10-20 at 11 43 17 AM
Screenshot 2023-10-20 at 11 43 31 AM

Another scenario also requires a fix.

Screen.Recording.2023-11-20.at.11.18.43.mov

Acceptance Criteria

The title or sort order or any folder value should be able to be updated and Push Publish.

Any folder values must be updated when the folder is moved on the receiver.

dotCMS Version

22.03

Proposed Objective

Technical User Experience

Proposed Priority

Priority 2 - Important

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

https://dotcms.slack.com/archives/CSHTYUR7H/p1697821222309339
https://dotcms.zendesk.com/agent/tickets/113652
https://dotcms.zendesk.com/agent/tickets/113708

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

@AndreyDotcms
Copy link
Contributor

AndreyDotcms commented Nov 14, 2023

Can't be reproduced on master

Screen.Recording.2023-11-14.at.12.09.03.mov

@AndreyDotcms
Copy link
Contributor

The issue was fixed on 23.01. But was reproducible in 22.03.11.

This issue was produced by an incorrect parameter order when copying the folder properties. This issue was fixed in EE.
The PR that fixed it https://github.com/dotCMS/enterprise/pull/903

@erickgonzalez erickgonzalez added LTS : Next Ticket that will be added to LTS LTS: Excluded Ticket that has been excluded from at least one LTS labels Nov 20, 2023
@erickgonzalez
Copy link
Contributor

Only applies to 22.03 LTS

@erickgonzalez erickgonzalez added Release : 22.03.12 Included in LTS patch release 22.03.12 and removed LTS : Next Ticket that will be added to LTS labels Dec 4, 2023
erickgonzalez added a commit that referenced this issue 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 : Customer Support Owned by Scott Priority : 2 High Release : 22.03.12 Included in LTS patch release 22.03.12 Team : Bug Fixers Type : Defect
Projects
Archived in project
Development

No branches or pull requests

4 participants