-
Notifications
You must be signed in to change notification settings - Fork 84
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
PSR-7 support #34
Comments
I'm working in a PSR-15 middleware for this here: https://github.com/oscarotero/matomo-tracker |
That sounds good, I presume we could potentially ship both the old (for BC) and new version |
@oscarotero awesome 👍! My intentions where never to propose a full rewrite but rather a polish up to the latest PSR's without doing a BC break. Maybe it's an idea to keep the piwik one as is and do a rewrite/polish up name matono? |
Great.
There's are many changes that need to be done and adding some tests would be nice. I think a new |
Why is it the best choice? I would have thought keeping it in here for best user experience, easy upgrade, easy manage of existing future issues etc in one repository maybe. |
Because the current package is registered as |
Nice work @oscarotero I was about to do what you've already done. 😄 Thumbs up for your implementation to become the new official implementation. |
Any updates on this? |
Are there any plans for PSR-7 support? It would be great if we could set up tracking and then get a PSR-7 request from the tracker to send it out using our own HTTP client. Also creating a new tracker from a PSR-7 server request would be appreciated. If you're open for this I gladly make both PR's
The text was updated successfully, but these errors were encountered: