Skip to content
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

Improve Start and end date for Tags handling of timezone #511

Closed
Starker3 opened this issue Jul 13, 2022 · 8 comments · Fixed by #538
Closed

Improve Start and end date for Tags handling of timezone #511

Starker3 opened this issue Jul 13, 2022 · 8 comments · Fixed by #538
Assignees
Labels
Bug Something isn't working

Comments

@Starker3
Copy link

We had a user that was having issues when configuring a tag with a specific start date and end date.

While investigating this issue, I found that the Start and End time for a Tag that is stored in UTC will be treated as that time in any timezone and not be converted to that visitors timezone.

For example, if we create a Tag such as the following:
Thursday-14-07-2022 11-24-15
In the above example the start date is set as 1 AM July 14 NZST with end date 8 PM July 14 NZST
When this is converted to UTC, the start date is 1 PM July 13 UTC with end date 8 AM July 14 UTC

When we then test this tag in the browser, the tag does not fire with the following in the logs:
Thursday-14-07-2022 11-24-28
not firing as this tag does not match date
However, the time of my device when testing was 11:20 AM NZST or 11:20PM UTC

When I changed the End Date of the tag to one day further (8 PM July 15 NZST, 8 AM July 15 UTC), it fired correctly as it detected the End Date time had not occurred yet:

Thursday-14-07-2022 11-26-01

This indicates that the time for the start/end date is converted to UTC but is used in the browser as the local timezone of the browser.

@AltamashShaikh
Copy link
Contributor

@Starker3 I have checked earlier versions too before VUE migration and its not a regression, we convert and store in UTC and post that we check if the visitor current time is in between the converted start and end datetime

@tsteur Just confirming this is how it is supposed to be right ?

@Starker3
Copy link
Author

@AltamashShaikh
I think this behavior has been the same since it was implemented. But the way it works currently would basically mean the tag would fire for some visitors some of the time.

@AltamashShaikh
Copy link
Contributor

@Starker3 Yes, its the same may be we should convert the visitors local time to UTC to do the comparison and it will be consistent for everyone, as currently the tag will fire/not fire depending upon visitors timezone and it will give a wrong impression on data.

@AltamashShaikh AltamashShaikh added this to the For Prioritisation milestone Jul 15, 2022
@Starker3
Copy link
Author

Starker3 commented Aug 4, 2022

@AltamashShaikh Do we know yet if this issue is going to be worked on?

@justinvelluppillai
Copy link
Contributor

@Starker3 We haven't yet prioritised it to be done. If you have other users waiting for this it would be good to let us know in this issue also

@Starker3
Copy link
Author

Starker3 commented Aug 4, 2022

@justinvelluppillai We currently just have the one user that reported this issue waiting on a fix.
But with what the issue is, it likely impacts every user that makes use of this feature in the tag manager. But with how it is currently, they might not realise it until they look more closely at the data and investigate why some actions are tracked before & after the dates & times set in the tag manager.

@AltamashShaikh
Copy link
Contributor

@Starker3 Should be fixed with release of Matomo 4.12, we will convert the user datetime to UTC during visit and check if its in the specified period defined and fire accordingly.

@Starker3
Copy link
Author

Thanks @AltamashShaikh !

@justinvelluppillai justinvelluppillai changed the title Start and end date for Tags uses UTC time for all timezones Improve Start and end date for Tags handling of timezone Aug 30, 2022
@innocraft-automation innocraft-automation removed this from the Current sprint milestone Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants