-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Release v2.0.0 rc.10 #5016
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 v2.0.0 rc.10 #5016
Conversation
GraphQL subscriptions
…lk-actions fix: tag bulk actions copy
…onVersionNumbers chore: wrong version in migration 54 & 55 file
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
…-package (feat) Add envalid package
Signed-off-by: Alex Rattray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Alex Rattray <[email protected]>
…m in many places Signed-off-by: Alex Rattray <[email protected]>
Signed-off-by: Alex Rattray <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Eric Dobbertin <[email protected]>
Signed-off-by: Eric Dobbertin <[email protected]>
Signed-off-by: Eric Dobbertin <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
- Combine the main .env.example file with any .env.example files from custom plugins during setup - ignore blank lines - also format with shfmt - also fix one shellcheck lint error Signed-off-by: Peter Lyons <[email protected]>
…ByAccountIdQuery feat: add ordersByAccountId query
Signed-off-by: Eric Dobbertin <[email protected]>
Signed-off-by: Eric Dobbertin <[email protected]>
Meteor and Docker performance tweaks
feat: Use .env.example files from custom plugins
Signed-off-by: Erik Kieckhafer <[email protected]>
…gWithNoTags fix: add tagId check to guard against undefined
spencern
left a comment
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.
@jeffcorpuz Great work getting this started!!
A few things remain to be done:
- Bump the version in
package.jsonandpackage-lock.json - Add any outstanding breaking change notices to the release notes
- Update
CHANGELOG.mdto include the release notes. - Tag a reviewer and post your automated testing notes once you've run it through your own QA process
I think it's probably worth noting this PR: #4992 and the significant improvements we've seen to reload times + removal of reaction-cli
spencern
left a comment
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.
Release notes look good. 👍
spencern
left a comment
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.
We'll need to update the version of lodash.merge included. I'm not sure what this will take.
@rosshadden recently updated our package.json to include the latest version of lodash 4.17.11, but it appears that the version of lodash.merge which is published separately to npm has not been updated and still contains a recently found low-severity vulnerability.
https://www.npmjs.com/package/lodash.merge
Vulnerability details: https://snyk.io/vuln/SNYK-JS-LODASHMERGE-173732
|
I started making PRs to the relevant repos: longshotlabs/js-message-box#10 |
Signed-off-by: Spencer Norman <[email protected]>
spencern
left a comment
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.
LGTM! 👍
v2.0.0-rc.10
This is our tenth release candidate for v2.0.0 of Reaction.
Please check it out and let us know what works and what doesn't for you.
This release is being coordinated with
reaction-platformand is designed to work with the same versions ofreaction-next-starterkitandreaction-hydra.Improvements
UI Removal
We have removed several UI components to transition and solidify that in 2.0 the application will only be used as an API and a UI for shop operators.
Additional PRs will be coming to remove other pieces of the storefront UI bit by bit until only an operator UI is left. (#4947 , #4948)
Performance Tweaks
reaction-cliwith this update. (Meteor and Docker performance tweaks #4992)General
localhost:3000/graphiqltolocalhost:3000/graphql-alphaGraphQL
Custom Data
calculateOrderTaxesto return a custom data object. (feat: allow tax services to add custom data to taxes #4955)Custom Plugins
Breaking Changes
AppEvents
Hooksusage with the newerappEvents. This does not break anything within the core and included plugins, however:If you use community or custom plugins that depend on the
@reactioncommerce/hookspackage, you will need to update or obtain updated versions that usecontext.appEventsinstead.If you have a plugin that uses
MethodHooks, update it to implement those hooks a different way.review all
appEventsconsumed and emitted by custom plugins. Update expected and emitted arguments. See the table. (Replace all Hooks with appEvents #4915)Multiple Payment Support
placeOrder*GraphQL mutations provided by the built-in payment plugins are removed and replaced with a singleplaceOrdermutation which supports multiple payments. Any custom payment method plugins will break due to the removal ofcreateOrderinternal mutation. Look at all changes. (Rewrite placeOrder and support multiple payments for an order #4908)Surcharges
Updates
inventoryQuantitytoinventoryInStockinProductscollection, update if used in your codebase. (refactor: UpdateinventoryQuantityfield to beinventoryInStock#4930)Removals
New Commands
In relation to improving performance, we have added new debugging statements (#4992) so you can use the Node debugger while developing Reaction!:
Example Usage :
New Documents
We have added new documentation! :
reaction-platformhas not been fully tested or is compatible with Windows.(https://docs.reactioncommerce.com/docs/next/installation-reaction-platform)
(https://docs.reactioncommerce.com/docs/how-to-create-a-fulfillment-methods-plugin)
Feature
Fixes
NaNin rare circumstances based on Migrations (fix: Inventory is set toNaNin rare circumstances based on Migrations #4946)Refactor
inventoryQuantityfield to beinventoryInStock(refactor: UpdateinventoryQuantityfield to beinventoryInStock#4930)Chores
Contributors
Thanks, @rattrayalex-stripe for contributing to this release!