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

fix(pp) fixing edge push cases where content and folders from non-pushed hosts gets dragged along by dependency, and containers in a drafted state fail to publish #30380

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

wezell
Copy link
Contributor

@wezell wezell commented Oct 17, 2024

ref: #30370

This PR updates the FolderHandler and the ContentletHandler to ignore folders and contents that live on a host that does not exist in the target system.

It also insures that the working inode of any pushed container is the one that is just saved by the ContainerHandler when it is creating a container for the first time, which fixes a case where there are different live/working versions of a container on the sender and the sender only sends the live version.

Finally, it fixes the crappy Bundle Upload screen so now the dialog does not stay open the whole time while the bundle is being uploaded AND ingested.

…hed hosts gets dragged along by dependency, and containers in a drafted state fail to publish

ref: #30370
@github-actions github-actions bot mentioned this pull request Oct 17, 2024
3 tasks
@wezell wezell linked an issue Oct 17, 2024 that may be closed by this pull request
3 tasks
@wezell wezell requested a review from jdotcms October 17, 2024 16:37
@wezell wezell enabled auto-merge October 17, 2024 16:37
…hed hosts gets dragged along by dependency, and containers in a drafted state fail to publish

ref: #30370
…hed hosts gets dragged along by dependency, and containers in a drafted state fail to publish

ref: #30370
Copy link

Quality Gate failed Quality Gate failed

Failed conditions
D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Edge PP issues
2 participants