matomo-org / matomo-sdk-android

SDK for Android to measure your apps with Matomo. Works on Android phones, tablets, Fire TV sticks, and more!
BSD 3-Clause "New" or "Revised" License
386 stars 163 forks source link

Single event doesn't appear on Dashboard #468

Open nostromoo opened 5 months ago

nostromoo commented 5 months ago

Hello,

We are using the android Matomo SDK version 4.1.4. We have notice that when we log one single event on a dispatchInterval period, it doesn't appear in the dashboard, but when there are several, it works fine.

After analysing the code of the Matomo sdk, I notice that to report a single event, the SDK is doing a http GET request and if there are more than one a http POST request (see send() method in DefaultPacketSender).

I tried to fork the Matomo SDK and modify the code to ensure that we use the POST call all the time, and it works fine.

Could you fix the issue please? Or tell us if there is a problem in our integration?

Thanks.

hannesa2 commented 5 months ago

When you fixed it, it would be fine, when you make a pull request for it

nostromoo commented 5 months ago

I have an error when I try to push my branch :

remote: Permission to matomo-org/matomo-sdk-android.git denied to nostromoo. fatal: unable to access 'https://github.com/matomo-org/matomo-sdk-android.git/': The requested URL returned error: 403

I tried with username/password and accessToken login

hannesa2 commented 5 months ago

You are not allowed to push to this repository, you need first an own fork. image Add this as a new remote to your repository and push it there. With this you can make a pull request.

Here you have the long and complete story https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests

nostromoo commented 5 months ago

Thank you for your reply, it's done : https://github.com/matomo-org/matomo-sdk-android/pull/470