-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Release 1.4.1 #2583
Merged
Merged
Release 1.4.1 #2583
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Init second request on after next url returned * Refactor to fetch carriers recursively from api
…-safari Remove isHidden check from core startup b/c blocking safari
…n-1921 Orders Pagination
* Remove revision conditional * Remove templates
* fix improper word wrap in product grid * fix improper product title wrap in firefox
* Don't pass in a cartID, pass in orderId for SalesInvoice * Don't try to parse lines unless we know we have it. No lines, no tax.
* Apply limit to products publication selector for admin users * Get correct top level products from publications * Set correct scroll check to load more products
Fixes console error ``` Error: You may not observe a cursor with {fields: {_id: 0}} ``` Not really sure when it got introduced, perhaps in Meteor `1.5.1rc5`? @erik and I had both seen this issue intermittently
- Release 1.3.1 renamed to 1.4.0 as there as some potential breaking changes in the new React components (PDP Grid, ReactTables replacement of Griddle).
* LingoHub Update 🚀 Manual push by LingoHub User: Aaron Judd. Project: reaction Made with ❤️ by https://lingohub.com * LingoHub Update 🚀 Manual push by LingoHub User: Spencer Norman. Project: reaction Made with ❤️ by https://lingohub.com
* longer timeout for circleci to run build.sh * update no_output_timeout with unit of time
* Adds inline validation to form variant form * remove commented code * Save data only if form is valid * move validation to a seperate plugin * Simplify passing validation object into components Adds ability to pass an entire validation status object into TextField component, and with a provided name, get the validation messages if available. * fixed card open / close states * Add validation to child variant form * Fixed variable name typo * Fixed typos on comment * Fixed broken checkbox update handler * Fixed top variant card closing on edit of some fields * Fix issues with validation and other fields - Changed variant form and child variant forms are now validaded on initial render. - Changed flash top varient field red if error, green if valid - Added `fields` object to validationStatus with field validation status and value - Added helper function to validation status to test field validity * Remove unused import
* Add instagram login/registration * Add browser policy * Fix user name not showing on navbar * Add styling to instagram button * browser policy for instagram images * add instagram translation * sign in formatting * Display user's profile name * revert padding, was applying to incorrect location * update browser policy
# Conflicts: # .meteor/packages # .meteor/versions # client/modules/accounts/helpers/util.js # client/modules/accounts/templates/profile/profile.js
Fix prop type value of MultiSelect to match values expected by underlying component.
Resolves #2333 Apparently the failing bcrypt / fallback installation was causing ..module not found errors. — need to evaluate for performance issues (see warning) — need to update docs to warn about the warning.. — todo find better solution!
brent-hoover
approved these changes
Jul 23, 2017
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed and ran smoke tests. Looks good to go.
aaronjudd
approved these changes
Jul 24, 2017
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved.
…eactively (#2561) * Reactively display only visible products for both admin and non-admin * Admin can see all products, non-admins can see only visible ones * Reactively publish new products * improve query to fix failing tests * Remove unnecessary react lifecycle method * Clean up products selector
Manual push by LingoHub User: Spencer Norman. Project: reaction Made with ❤️ by https://lingohub.com
Withheld 2561 for now.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes / Fixes