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
(chore) #1364 rename Import to Importer #3613
(chore) #1364 rename Import to Importer #3613
Conversation
…of a reserved keyword.
…of a reserved keyword.
…tps://github.com/reactioncommerce/reaction into refactor-1364-willopez-rename-import-to-importer
lgtm |
Once this PR is merged, please update https://docs.reactioncommerce.com/reaction-docs/master/reaction-import in the docs with this ticket: reactioncommerce/reaction-docs#491 |
…of a reserved keyword.
…tps://github.com/reactioncommerce/reaction into refactor-1364-willopez-rename-import-to-importer
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.
Looks good.
Seeing this warning inside the tests, but all tests pass and this is not specific to this PR:
W20180222-10:42:59.216(-8)? (STDERR) (node:40145) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 152): Error: Method 'reaction/getUserId' not found [404]
@willopez I think the tests broke when I merged 1.8.2 in to this branch to get it ready to release. Let's pair on it tomorrow am and get it ready for 1.9 |
@spencern sure, just ping me. |
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.
Fixed after issue from merge conflict.
@@ -330,7 +330,7 @@ export const methods = { | |||
Meteor.call("orders/shipmentDelivered", order); | |||
} | |||
|
|||
// A batch update might be better option. Unfortunately Reaction.import doesn't support | |||
// A batch update might be better option. Unfortunately Reaction.importer doesn't support |
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.
This should be capitalized Reaction.Importer
as Reaction.importer
doesn't exist.
Resolves #1364
Impact: breaking
Type: refactor
Issue
Rename Import module to Importer. This prevents the usage of a reserved keyword.
Solution
server/api/core/import.js
toserver/api/core/importer.js
Testing
core/layout, core/taxes,included/shipping-shippo, included/taxes-taxcloud, core/core(settings), core/temaplates.js, server/startup/load-data.js,