chore: Track edit events

[AshAman999]

What

• Add Matomo events for tracking edits
• Complete

Part of

• Add events for each section of the Edit mode in Matomo #3547

[M123-dev]

Heyy @AshAman999 sorry for my somewhat delayed review, the core looks great you've managed to make it very compact, thats great

[teolemon]

Great, It would really be nice to get this in the next version 👍

[AshAman999]

Edit:
Complete and ready to review

[monsieurtanuki]

Though I think i was able to handle most of the event trackings , i think i am getting confused in the simple_input_page.dart.

I am not able to find a correct way as of today for getting the type of edit from within the page itself , added a TODO for the same in packages\smooth_app\lib\pages\product\simple_input_page.dart Line 185 , would seek some help here 😅

Hi @AshAman999! There's no "better way" to get the edit type from the page. It's either "there's just one type involved" or "there are several types involved". In both cases it's final List<AbstractSimpleInputPageHelper> helpers.
So helpers is the solution you're looking for. Does that fit the types you expect for the track edit, that's another story.

[AshAman999]

Though I think i was able to handle most of the event trackings , i think i am getting confused in the simple_input_page.dart.
I am not able to find a correct way as of today for getting the type of edit from within the page itself , added a TODO for the same in packages\smooth_app\lib\pages\product\simple_input_page.dart Line 185 , would seek some help here 😅

Hi @AshAman999! There's no "better way" to get the edit type from the page. It's either "there's just one type involved" or "there are several types involved". In both cases it's final List<AbstractSimpleInputPageHelper> helpers. So helpers is the solution you're looking for. Does that fit the types you expect for the track edit, that's another story.

Figured out the solution, thanks for the resuce @monsieurtanuki 😄

[monsieurtanuki]

@AshAman999 @M123-dev How come?

[M123-dev]

That is long 🫣
Just force-canceled it, but I guess this is some kind of bug, I'm sure they have timeouts in place

[monsieurtanuki]

Hi @AshAman999!
I haven't followed the issue from the beginning, but except my 2 suggestions and the fact that you don't like trailing commas, that's good for me!

[AshAman999]

Seems like the tests gonna run foreover again, @M123-dev ,
Manually cancelling the workflow for now