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

refactor: use npm package for shops plugin #6201

Merged
merged 3 commits into from
Apr 15, 2020

Conversation

mpaktiti
Copy link
Contributor

Resolves #6180

This PR removes the internal shops plugin, and installs the new api-plugin-shops from npm as a replacement.

The code in that npm plugin is the same, so the expected outcome of this PR is seeing nothing different than how the app currently runs.

Testing

  • Start reaction, see that shops is the correct version in the systemInformation query
  • Test the shops queries/mutations and see that they work

Signed-off-by: Maria Paktiti [email protected]

mpaktiti and others added 2 commits April 15, 2020 15:29
Signed-off-by: Maria Paktiti <[email protected]>
Signed-off-by: Erik Kieckhafer <[email protected]>
Copy link
Member

@kieckhafer kieckhafer 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

@kieckhafer kieckhafer merged commit 8e26649 into trunk Apr 15, 2020
@mpaktiti mpaktiti deleted the refactor-mpaktiti-move-shops-plugin branch April 15, 2020 23:05
@kieckhafer kieckhafer mentioned this pull request Apr 16, 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.

move shops plugin to its own npm package, and install npm version Reaction
2 participants