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
Staticize facade and complete API actions #15
Staticize facade and complete API actions #15
Conversation
These changes conform this package to follow the "Laravel Way". The API actions wrapped by the Coinbase facade are now truly staticized. Invoice management functionality has been implemented and missing actions for Charges (cancel and resolve) have also been implemented. Readme has been updated commensurately with changes made to the codebase.
@antimech @pierre-pizzetta , any word from @shakurov ? He seemed to have taken an extended vacation from this package. |
@alexstewartja no answers from him. I'll take a look at your PR. |
LGTM but the merge conflict must be fixed and it seems like you haven't ticked the "allow edits from maintainers", |
"Allow edits" enabled |
🚀 |
I don't have access to the repository settings and can't enable packagist auto-update. Your changes are available in antimech/coinbase. I recommend switching to my repository from now on: composer remove shakurov/coinbase
composer require antimech/coinbase |
Seems to have been (manually) updating quite fine until today: https://packagist.org/packages/shakurov/coinbase What went wrong? |
Manual updates are under big delay. Plus I have to wait until I can request it. |
I've updated it, manual updates available once in 24 hours. I'll support this package for some time for graceful replacement. |
These changes conform this package to follow the "Laravel Way".
Changes
The API actions wrapped by the Coinbase facade are now truly staticized.
Invoice management functionality has been implemented and missing actions for Charges (cancel and resolve) have also been implemented.
Readme has been updated commensurately with changes made to the codebase.
Checklist