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

updated giftcard.js #27

Merged
merged 1 commit into from
Oct 20, 2016
Merged

updated giftcard.js #27

merged 1 commit into from
Oct 20, 2016

Conversation

stevebosworth
Copy link
Contributor

@Shopify/themes-fed

  • move giftcard.js into it's own file and load it on the giftcard template
  • do not load theme.js on giftcard template

demo: https://checkout.shopify.com/gift_cards/8984760/77bb8c5fd0b314402f092280c388e17c

@stevebosworth stevebosworth self-assigned this Oct 20, 2016
@bertiful
Copy link
Contributor

👍

@bertiful
Copy link
Contributor

Apart from Unexpected template string expression no-template-curly-in-string.

@bertiful
Copy link
Contributor

Oh wait, that should be fixed with the PR you pushed earlier, no? (I'll stop spamming now)

Copy link
Contributor

@cshold cshold left a comment

Choose a reason for hiding this comment

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

Remove gift card JS from theme.js too

@@ -4,7 +4,7 @@
* A file that contains scripts highly couple code to the Gift Card template.
*/

theme.giftCard = (function() {
(function() {
if (!$('body').hasClass('template-giftcard')) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can remove this check

@stevebosworth stevebosworth merged commit 38e90df into master Oct 20, 2016
@stevebosworth stevebosworth deleted the move-giftcard-js branch October 20, 2016 19:36
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
* updates dependencies

* updated eeeeerrrrything
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
* updates dependencies

* updated eeeeerrrrything
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
* updates dependencies

* updated eeeeerrrrything
@lock
Copy link

lock bot commented Oct 26, 2018

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 Oct 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants