Skip to content
This repository has been archived by the owner on Jul 19, 2021. It is now read-only.

Fix lax checking of filename against genericTemplateName in sections plugin #1008

Merged
merged 2 commits into from
Mar 26, 2019

Conversation

andyexeter
Copy link
Contributor

@andyexeter andyexeter commented Mar 14, 2019

The sections plugin was performing a lax check on the section filename against the genericTemplateName option, using the includes() method to see if the relativePathFromSections argument includes the genericTemplateName option value anywhere within it.

This meant files such as src/sections/collection-template.liquid were compiled to dist/sections/collection-template.liquid.liquid and were not being updated on Shopify when changes were made locally with yarn start running.

This fix compares the basename of the relativePathFromSections argument instead.

Checklist

For contributors:

For maintainers:

  • I have 🎩'd these changes.

@ghost ghost added the cla-needed label Mar 14, 2019
@andyexeter
Copy link
Contributor Author

I've signed the CLA - not sure what I need to do to get the check to run again.

@ghost ghost removed the cla-needed label Mar 19, 2019
Copy link
Contributor

@bertiful bertiful left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your PR @andyexeter :)

@Maurice-JB
Copy link

Just checking -- what is the status of this PR? I can see it's been merged into Master but running
yarn upgrade --latest @shopify/slate-tools
doesn't seem to update the package.json or update any dependencies...

@wizardlyhel
Copy link
Contributor

@Maurice-JB Working on it (This week). I'm fiddling with auto deploy configurations.

@lock
Copy link

lock bot commented May 8, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants