Skip to content

Change case of Class Name WoocommerceApi => WooCommerceApi. #14

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

Merged

Conversation

rub3n88
Copy link

@rub3n88 rub3n88 commented Mar 30, 2020

Hello!
This is my first "pull request". Sorry if I don't do it right.
The package generates the following error on my production server:
  "Class Codexshaper \ WooCommerce \ WooCommerceApi does not exist",

Reviewing, I see that the class name really is WocommerceApi.
I see that in the "Register Provider" there is an alias, but still I have not got it to work.

Finally I have chosen to fork and rename the class and the file name.

If it is correct, I would appreciate if you accept the "pull request" to reconnect my project directly to your repository and enjoy the next updates.

If I am wrong, I would also appreciate if you could explain how to solve it.

Thank you.

In Production server not found the class.
@Codexshaper Codexshaper merged commit 54c6678 into Codexshaper:master Mar 30, 2020
@Codexshaper
Copy link
Owner

@rub3n88 Merged your request.

@rub3n88
Copy link
Author

rub3n88 commented Mar 30, 2020

Thanks!!

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.

2 participants