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

[Bug]: shopify theme share uploads files ignored in .shopifyignore #2664

Closed
2 tasks done
hsdonkin opened this issue Aug 17, 2023 · 6 comments · Fixed by #4578
Closed
2 tasks done

[Bug]: shopify theme share uploads files ignored in .shopifyignore #2664

hsdonkin opened this issue Aug 17, 2023 · 6 comments · Fixed by #4578
Assignees
Labels
Area: @shopify/theme @shopify/theme package issues #gsd:40767 Fortify local development experience for Liquid themes important second highest severity for theme related bug Type: Bug Something isn't working

Comments

@hsdonkin
Copy link

Please confirm that you have:

  • Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
  • Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?

Theme

Expected behavior

When uploading a theme via shopify theme share, files included in the .shopifyignore should not be uploaded

package.json
package-lock.json
node_modules
node_modules/

Actual behavior

Ignored files are uploaded, and upload hangs due to number of files in node_modules 😅
image

Reproduction steps

  1. Install any npm packages, i.e. npm init && npm i preact
  2. Add package.json and package-lock.json and node_modules to .shopifyignore
  3. shopify theme share

Operating System

OSX 11.5.1

Shopify CLI version (check your project's package.json if you're not sure)

3.4.2

Shell

No response

Node version (run node -v if you're not sure)

No response

What language and version are you using in your application?

No response

@hsdonkin hsdonkin added the Type: Bug Something isn't working label Aug 17, 2023
@isaacroldan isaacroldan added the Area: @shopify/theme @shopify/theme package issues label Aug 21, 2023
@github-actions
Copy link
Contributor

This issue seems inactive. If it's still relevant, please add a comment saying so. Otherwise, take no action.
→ If there's no activity within a week, then a bot will automatically close this.
Thanks for helping to improve Shopify's dev tooling and experience.

P.S. You can learn more about why we stale issues here.

@beelarr
Copy link

beelarr commented Sep 13, 2023

I am running into a related issue where using theme share in CI throws this error. package.json: package.json is not a theme file. Our .shopifyignore contains the package.json so it seems like theme share is ignoring the .shopifyignore

@shopify/cli/3.44.1 darwin-arm64 node-v19.8.1

@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2023

This issue seems inactive. If it's still relevant, please add a comment saying so. Otherwise, take no action.
→ If there's no activity within a week, then a bot will automatically close this.
Thanks for helping to improve Shopify's dev tooling and experience.

P.S. You can learn more about why we stale issues here.

@stevehimmself
Copy link

I am experiencing the same issue with Shopify.

@johnphillipsdev
Copy link

Running into the same issue, moving the theme files into its own directory is helpless too as Shopify can't find them via subdirectories on the store connection.

@karreiro
Copy link
Contributor

karreiro commented Oct 4, 2024

Thank you for reporting this issue, @hsdonkin. It's being solved by #4578.

@karreiro karreiro linked a pull request Oct 4, 2024 that will close this issue
5 tasks
@karreiro karreiro added important second highest severity for theme related bug #gsd:40767 Fortify local development experience for Liquid themes labels Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: @shopify/theme @shopify/theme package issues #gsd:40767 Fortify local development experience for Liquid themes important second highest severity for theme related bug Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants