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

Use packagist for module installation #21

Closed
wants to merge 32 commits into from

Conversation

fredden
Copy link

@fredden fredden commented Dec 7, 2020

This package is available via packagist. https://packagist.org/packages/mailplus/mailplus-connector

Paul Bosselaar and others added 30 commits August 24, 2017 14:22
The healthcheck is used for getting the Plugin ver
The resource `Magento_Sales::order` that was used isn't a valid
resource which results in the connection to that route
(/V1/mailplus/orders) to be refused. The correct resource is
`Magento_Sales::sales` which is also used for the "normal" routes
regarding the orders.
…-resource-for-mailplus-order

Fix incorrect resource for webapi
@fredden
Copy link
Author

fredden commented Feb 1, 2023

@808brinks why was this closed?

@808brinks
Copy link
Contributor

Seems to have automatically closed because of a sync I've setup for moving this project from our local gitlab to github. Is it possible to open a new pull request?

The new master doesn't have the same git history as the old master.

@fredden
Copy link
Author

fredden commented Feb 1, 2023

Thanks for the quick feedback. I've rebased the changes here and created #33 as suggested.

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.

5 participants