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

chore: updated api plugins #6265

Merged
merged 7 commits into from
Jun 30, 2020
Merged

chore: updated api plugins #6265

merged 7 commits into from
Jun 30, 2020

Conversation

rosshadden
Copy link
Contributor

Updates API plugins.

Signed-off-by: Ross Hadden <[email protected]>
@rosshadden rosshadden requested a review from mpaktiti June 23, 2020 15:38
package.json Outdated
"@reactioncommerce/api-plugin-pricing-simple": "~1.0.1",
"@reactioncommerce/api-plugin-products": "~1.0.1",
"@reactioncommerce/api-plugin-settings": "~1.0.1",
"@reactioncommerce/api-plugin-shipments-flat-rate": "~1.0.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

@rosshadden The problem is with this line, if I set it to "@reactioncommerce/api-plugin-shipments-flat-rate": "~1.0.0" locally it passes the failing tests.

We can either wait until we fix this, or rollback this update only and merge the rest.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good find! I bet it was time-consuming. I'll just change this PR to keep that package at the version you found to work.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mpaktiti I tried both ~1.0.0 and 1.0.0, but the tests are still failing. Can you play with making your changes to this branch and try to get it working in CI?

Copy link
Contributor

Choose a reason for hiding this comment

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

I set it to 1.0.0 and rebuilt the lock file, tests are passing now ¯_(ツ)_/¯

I'll register an issue at the plugin's repo (with a note to update this repo's package.json) and look into it.

@mpaktiti mpaktiti self-requested a review June 26, 2020 12:52
mpaktiti
mpaktiti previously approved these changes Jun 26, 2020
@mpaktiti
Copy link
Contributor

Issue for the broken dependency reactioncommerce/api-plugin-shipments-flat-rate#5

Signed-off-by: Mike Murray <[email protected]>
Signed-off-by: Mike Murray <[email protected]>
@mikemurray
Copy link
Member

mikemurray commented Jun 29, 2020

@mpaktiti I added fixes, some to this PR, one for shipments based on your ticket, and one for API utils which should address some, hopefully all of broken integration tests.

Signed-off-by: Maria Paktiti <[email protected]>
@mpaktiti
Copy link
Contributor

Looks good @mikemurray, thanks! I updated both packages to latest and all tests pass.

cc @rosshadden @manueldelreal

@mpaktiti mpaktiti self-requested a review June 30, 2020 12:12
Copy link
Member

@manueldelreal manueldelreal left a comment

Choose a reason for hiding this comment

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

My automated tests pass with this one

@mikemurray mikemurray merged commit bd4799f into trunk Jun 30, 2020
@mikemurray mikemurray deleted the chore-update-deps branch June 30, 2020 20:42
@kieckhafer kieckhafer mentioned this pull request Aug 4, 2020
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.

4 participants