Update of Matomo Device Detector

[arvinv]

Hey Etienne,

Thanks for this NodeJs port of Matomo Device Detector.

So I have two questions ?

1. How do we ensure that this port has all the new changes from the Matomo Device Detector ? Is there a way to manually trigger the update ?

2. Looks like Matomo has released a new minor version 3.11.2. Do you need to update your fork of the PHP Matomo Device Detector library to get the latest changes ?

https://github.com/matomo-org/device-detector/releases

Regards,
Arvin

[etienne-martin]

Hi @arvinv,

I periodically update my fork of matomo's device-detector (https://github.com/etienne-martin/matomo-device-detector) on which this lib depends.

Here's what I need to do in order to make my lib up-to-date with matomo's lib:

1. Update the https://github.com/matomo-org/device-detector#{version-number} to the latest version of matomo device detector in the package.json
2. run yarn fixtures to update to the latest fixtures
3. Run yarn test and update the library if some tests are failing
4. Open a PR

Maybe I forgot something, last time I updated it was Sep 17, 2018. I should definitely write some documentation about the process.

Thanks.

[arvinv]

Thanks for the information.

[etienne-martin]

@arvinv I just ugraded the lib to 3.11.2

[whimsicaldreamer]

Hi @etienne-martin matomo device detector has upgraded to 3.11.7. Will you be updating it soon?

[etienne-martin]

Ideally yes, but I'm pretty swamped right now. Feel free to follow these steps and open a PR.

[whimsicaldreamer]

I am not sure but the fork https://github.com/etienne-martin/matomo-device-detector gives 404 to me.

[whimsicaldreamer]

@etienne-martin @AlexBeauchemin I was hoping if you could guide me on the way to update this library. I have forked the original matomo device detector library and also this library. Then I created a branch from master of the original library and then updated the napa field with the branched URL. Ran npm install and also npm run test. All tests seemed to pass but how do I understand if this library is really updated?

[etienne-martin]

Hi @whimsicaldreamer, things changed since I posted the original instructions.

Here are the up-to-date instructions on how to update the library:

1. Pull the master branch from this repo
2. Update the https://github.com/matomo-org/device-detector#{version-number} to the latest version of matomo device detector in the package.json
3. run yarn fixtures to update to the latest fixtures. This will download the fixtures based on the version specified in the package.json
4. Run yarn test and update the library if some tests are failing
5. Open a PR

You can also skip those steps and fork this branch that is up-to-date with version 3.12.1 of matomo but with a bunch of failing tests:

It should be fairly easy to fix them. It's usually due to new devices that were added to https://github.com/matomo-org/device-detector/blob/1060530615c54796cc2045a030c7c605574060d2/Parser/Device/DeviceParserAbstract.php#L65 that needs to be added to this file.

Let me know how that goes and don't hesitate to ping me on twitter @_etiennemartin if you want a quicker response.

Thank you 🙏

[etienne-martin]

Well, no need to do anything, I just released v2 which is up-to-date with matomo. Thanks for offering help tho

[whimsicaldreamer]

@etienne-martin I was just starting to work on it, but its great to see the new release. :) I will keep track if new releases are made and will try to PR the changes, so that this repo is up to date. Thank you for your efforts.

[whimsicaldreamer]

Btw, in the readme, https://github.com/etienne-martin/device-detector-js#why-are-the-tests-failing-after-downloading-new-fixtures, "this file" link is dead. And also https://github.com/etienne-martin/device-detector-js#this-is-a-javascript-port-of-matomo-device-detector-3114, the version number needs to be updated to the recent one.

[etienne-martin]

Thanks a lot, I did fix the version number but didn't notice the broken link, good catch!

[etienne-martin]

Should be fixed now, let me know if you find anything else and thanks again

[whimsicaldreamer]

Great! Will let you know in case if I find something more, or else I will do a PR.