Enable sidekick publish on files in `.helix/` about helix-sidekick HOT 7 CLOSED

@tripod pointed out that we don't really need to publish .helix/config.json and when merging we should just pull this from preview. So I've updated this issue to only allow publish operations on .helix/headers.json. That said I can see a world where in the future were potentially we want to enable publish on other files in .helix/.

from helix-sidekick.

So basically, only .helix/config.json should be treated as a special case?

cc @tripodsan

from helix-sidekick.

So basically, only .helix/config.json should be treated as a special case?

cc @tripodsan

for now, I would just disable publish on all /.helix. we can still open it up if needed.

from helix-sidekick.

OK, that's the current status. Then I don't understand this enhancement request. 😄

from helix-sidekick.

right, I forgot that there is no publish button on the edit urls. closing as won't fix.

from helix-sidekick.

Sorry, somehow missed this convo..

.helix/config-all.json is unique to each partition as per.
https://github.com/adobe/helix-html-pipeline/blob/main/src/steps/fetch-config-all.js#L103

If we are not going to enable publish we should do something similar to what is done with preview operations on .helix/config where somehow (haven't looked) a preview results in .helix/config-all.json in the live partition being updated.

Do you agree @tripodsan?

from helix-sidekick.

I think /.helix/headers.json should also not need a publish.

from helix-sidekick.