internalfx / quickshot

A Shopify theme development tool.
ISC License
107 stars 18 forks source link

Changes to checkout.liquid are not being pushed up. #75

Open joshuaiz opened 4 years ago

joshuaiz commented 4 years ago

Hello,

Still loving Quickshot.

I'm using it with a Shopify Plus client and it seems that any changes to checkout.liquid are not being pushed up to the target store. All other files are fine including theme.liquid.

Is there anything that could be blocking changes to checkout.liquid or otherwise preventing these changes to be pushed up?

My best guess is it is some sort of permissions issue.

Here are the steps to reproduce:

  1. Create target
  2. Use theme watch --sync command
  3. Make local changes to checkout.liquid
  4. Save file; Terminal reports: 02:48:25 pm - Added/Updated theme/layout/checkout.liquid
  5. Reload Checkout page and/or check in Themes > [theme] > Edit theme code
  6. checkout.liquid file is not updated.

If you need more information, let me know. Thanks!

internalfx commented 4 years ago

I'm looking at the code right now and I can't see any reason it wouldn't work

internalfx commented 4 years ago

Can you try using theme upload directly?

Let me know...

joshuaiz commented 4 years ago

I just tried using this:

$ qs theme upload theme/layout/checkout.liquid

and it started to upload every file in the theme(!!!!) What am I missing here?

I only have 1 target configured.

internalfx commented 4 years ago

You want to use the --filter option

joshuaiz commented 4 years ago

@internalfx thanks.

Here's what I tried:

$ qs theme upload --filter checkout.liquid
11:04:28 pm - Uploaded 0 files.
$ qs theme upload --filter theme/layout/checkout.liquid
11:05:08 pm - Uploaded 0 files.
$ qs theme upload --filter /theme/layout/checkout.liquid
11:05:37 pm - Uploaded 0 files.
$ qs theme upload theme/layout/checkout.liquid --filter 
11:07:26 pm - Uploaded 0 files.

Doesn't seem like it's uploading or I'm doing something wrong.

internalfx commented 4 years ago

qs theme upload --filter checkout.liquid should have worked....

joshuaiz commented 4 years ago

@internalfx yep it should have but it doesn't as you can see above: 'Uploaded 0 files.'

My best guess is it is a permissions issue since checkout.liquid is only available to Shopify Plus stores so maybe there are different permissions for that?

I looked at my private app settings and couldn't see anything that jumped out at me. I can follow up also with Shopify Plus support.

internalfx commented 4 years ago

I unfortunately don't have a Shopify plus account I can test this with.

Please do let me know what you find out.

For reference the API call I'm using is this.

https://shopify.dev/docs/admin-api/rest/reference/online-store/asset?api[version]=2020-04#update-2020-04 image

joshuaiz commented 4 years ago

Sorry for the delay on this - I contact Shopify Plus support who elevated the issue to their dev team and this is what they just came back with:


I've chatted with our partner support team about the issues that you are having with Quickshot not publishing updates to your checkout.liquid. At this time, they have advised that it would be best if the developer of the app reached out directly through their partner dashboard for more information on this.

If you don't already have a Shopify partners account, you can create one here: https://www.shopify.com/partners

Hopefully they can give you some more information directly.

I can confirm that this is still an issue so any changes to checkout.liquid have to be made in the Shopify admin and don't get pushed up via Quickshot. Not an emergency by any means but I would assume other people are having the same issue.

internalfx commented 2 years ago

@joshuaiz has anything changed regarding this issue?

joshuaiz commented 2 years ago

This issue still remains - pushes to checkout.liquid do not work with Quickshot.

internalfx commented 2 years ago

@joshuaiz ok, thank you, I'll take a deeper look.