Update base.rb,fixed matomo API call#33
Update base.rb,fixed matomo API call#33Code-Farmer-Ming wants to merge 4 commits intomatomo-org:masterfrom
Conversation
|
Thank you for the pull request. I think, loosening the ActiveSupport constraint is the right call. But why did you change the request method from GET to POST? Which problem does this solve? |
Hey! I was just checking out the gem, and found this PR. Based on matomo documentation, when using an |
|
Sounds plausible. If that is the case, personally I'd prefer to keep using GET and only use POST when an Anyway, I no longer have merge authorization, so it doesn't actually matter... |
Description:
Fixed gem work with new version matomo, changed the API call from
gettopost