-
Notifications
You must be signed in to change notification settings - Fork 467
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
Push publishing a content type fails if it is on a site that does not exist on target AND NO Error Message #29255
Labels
dotCMS : Push Publishing
LTS: Needs Backport
Ticket that will be added to LTS
QA : Approved
QA : Passed Internal
Release : 24.09.21
UVE // Java21
Team : Falcon
Type : Defect
Comments
NOTE TO DEVS:When the error message is being put together: Lines 210 to 212 in 3870c53
The String.format method is expecting 3 parameters, but it's receiving 4. Therefore, the error message from the original exception is not going to be present in the message displayed in the UI.
|
wezell
added a commit
that referenced
this issue
Jul 24, 2024
wezell
added a commit
that referenced
this issue
Jul 24, 2024
wezell
added a commit
that referenced
this issue
Jul 24, 2024
github-project-automation
bot
moved this from In Review
to Internal QA
in dotCMS - Product Planning
Aug 20, 2024
Failed Internal QA: Saving the Content Type under default host instead of system host |
dsilvam
moved this from Internal QA
to Current Sprint Backlog
in dotCMS - Product Planning
Aug 26, 2024
dsilvam
moved this from Current Sprint Backlog
to In Progress
in dotCMS - Product Planning
Aug 27, 2024
dsilvam
added a commit
that referenced
this issue
Aug 27, 2024
github-merge-queue bot
pushed a commit
that referenced
this issue
Aug 28, 2024
The siteName is also needed to be set to the CT because otherwise the host it gets looked up by siteName first if it does not exist on the receiver it will fallback to defaultHost, which is *not* the desired behavior here, which is fallback to system_host.
Pass Internal QA: Now the Content Type is saved under the SYSTEN_HOST |
Approved: Tested on trunk_b8fe837, Docker, macOS 14.5, FF v126.0.1 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
dotCMS : Push Publishing
LTS: Needs Backport
Ticket that will be added to LTS
QA : Approved
QA : Passed Internal
Release : 24.09.21
UVE // Java21
Team : Falcon
Type : Defect
Parent Issue
No response
Problem Statement
If you create a content type that lives on host that does not exist on the target site, it will fail to publish and give no error. See screenshot below with no actionable error:
Steps to Reproduce
Acceptance Criteria
This would prevent failures.
At minimum, we should show a better error message that describes the actual problem.
The text was updated successfully, but these errors were encountered: