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

1762 + Meteor 1.4.2.7 #1855

Merged
merged 4 commits into from
Feb 20, 2017
Merged

1762 + Meteor 1.4.2.7 #1855

merged 4 commits into from
Feb 20, 2017

Conversation

aaronjudd
Copy link
Contributor

@aaronjudd aaronjudd commented Feb 17, 2017

  • Meteor 1.4.2.7
  • Resolves fix transliteration import #1762
  • Removes transliteration.min.js (import direct)
  • special case handling on PDP handle update
  • use alternate minifier to resolve broken builds.

Testing:
1.4.2.7:
running the app normally -> no issue should be caused by 1.4.2.7, but pay attention to module loading.

For transliteration:
Login in as admin, change your language to Chinese, Hebrew etc..
Save products handle, check that route name has been transliterated AND slugified..

Test with routes, by updating shop name, refresh, check routes, like pdp, are using a new transliterated, slugified route.

aaronjudd added 2 commits February 16, 2017 00:30
- Meteor 1.4.2.7
- updated dependencies
- updated yarn.lock
- Resolves #1762
- Removes transliteration.min.js (import direct)
- special case handling on PDP handle update
Copy link
Member

@mikemurray mikemurray left a comment

Choose a reason for hiding this comment

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

Works as expected.

Updated to use alternate minifier

abernix:standard-minifier-js

See Meteor Issue 8378
@aaronjudd aaronjudd requested review from jshimko and removed request for brent-hoover February 18, 2017 15:28
@jshimko jshimko merged commit 1508ce7 into development Feb 20, 2017
@jshimko jshimko deleted the meteor-1.4.2.7 branch February 20, 2017 17:23
@jshimko jshimko removed the review label Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants