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

Pushing sites from within the sites folder fails #28473

Closed
fabrizzio-dotCMS opened this issue May 7, 2024 · 3 comments
Closed

Pushing sites from within the sites folder fails #28473

fabrizzio-dotCMS opened this issue May 7, 2024 · 3 comments

Comments

@fabrizzio-dotCMS
Copy link
Contributor

Parent Issue

No response

Problem Statement

if you're working with sites and you cd into the sites folder make some changes and then call the global push from within the sites folder an error occurs like :

fabrizzioaraya@Fabrizzios-MacBook-Pro sites % dot push
/
 ──────
 No changes in Languages to push


|
 ──────
 Push Data: [1] Sites to push: (0 New - 1 Modified)


-███████████████████████████████████████████████████████████████████████████████████████████████████  100%
 ──────
 No changes in ContentTypes to push


[ERROR] ❗  Error in command [push] with message:
  Invalid source path. Source path must be inside the files folder or at the root of the workspace
run with -e or --errors for full details on the exception.

Steps to Reproduce

cd into the workspace/sites folder make some changes to any of the site json files Maybe add a site variable as

{
  "dotCMSObjectType" : "Site",
  "inode" : "2ba7223d-4b9a-4f33-b7d5-fa3e782ecb84",
  "identifier" : "8a7d5e23-da1e-420a-b4f0-471e7da8ea2d",
  "aliases" : null,
  "systemHost" : false,
  "siteName" : "default",
  "tagStorage" : "8a7d5e23-da1e-420a-b4f0-471e7da8ea2d",
  "siteThumbnail" : "",
  "runDashboard" : false,
  "keywords" : null,
  "description" : null,
  "googleMap" : null,
  "googleAnalytics" : null,
  "addThis" : null,
  "proxyUrlForEditMode" : null,
  "embeddedDashboard" : null,
  "languageId" : 1,
  "variables" : [ {
    "name" : "foo",
    "key" : "bar",
    "value" : "bar"
  } ],
  "modDate" : "2024-05-07T23:14:21.802+00:00",
  "modUser" : "dotcms.org.1",
  "live" : true,
  "default" : true,
  "locked" : true,
  "archived" : false,
  "working" : true
}
then call the global push from within the sites folder as follows 

user@user-MacBook-Pro sites % dot push

Acceptance Criteria

We should be able to call global any operation from with any folder within the workspace and the command should be able to find its way within the workspace

dotCMS Version

master

Proposed Objective

Customer Success

Proposed Priority

Priority 3 - Average

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

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

@fabrizzio-dotCMS
Copy link
Contributor Author

fabrizzio-dotCMS commented Aug 8, 2024

I think the changes introduced to improve handling relative paths
here could have fixed this problem
Sending it to IQA for verification

@fabrizzio-dotCMS fabrizzio-dotCMS moved this from Gardening Day Backlog to In Review in dotCMS - Product Planning Aug 8, 2024
@jgambarios jgambarios moved this from In Review to Internal QA in dotCMS - Product Planning Aug 8, 2024
@jgambarios jgambarios self-assigned this Aug 8, 2024
@jgambarios
Copy link
Contributor

Note to QA:

As Fabrizzio already mentioned, this was fixed in #28593
Passed Internal QA as I was unable to reproduce the issue.

@jgambarios jgambarios moved this from Internal QA to QA - Backlog in dotCMS - Product Planning Aug 8, 2024
@github-project-automation github-project-automation bot moved this from QA - Backlog to Internal QA in dotCMS - Product Planning Aug 8, 2024
@jgambarios jgambarios moved this from Internal QA to QA - Backlog in dotCMS - Product Planning Aug 8, 2024
@jgambarios jgambarios removed their assignment Aug 8, 2024
@bryanboza
Copy link
Contributor

Fixed unable to reproduce after the fix mentioned in the thread.

Tested on the last CLI // Docker

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

No branches or pull requests

5 participants