mautic
mautic copied to clipboard
Report - fix date data display for local timezone
| Q | A |
|---|---|
| Bug fix? (use the a.b branch) | [ ] |
| New feature/enhancement? (use the a.x branch) | [ ] |
| Deprecations? | [ ] |
| BC breaks? (use the c.x branch) | [ ] |
| Automated tests included? | [ ] |
| Related user documentation PR URL | mautic/mautic-documentation#... |
| Related developer documentation PR URL | mautic/developer-documentation#... |
| Issue(s) addressed | Fixes #... |
Description:
In some cases could date field make display wrong data due UTC conversion. We fixed it
Steps to test this PR:
- Open this PR on Gitpod or pull down for testing locally (see docs on testing PRs here)
- Create custom field date for contact
- Create some contacts with values in that field
- Create report with the list of contacts with the date field in the column
- Just check If date field is displayed in report like before
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 63.13%. Comparing base (
013e592) to head (5c42ece). Report is 53 commits behind head on 5.x.
Additional details and impacted files
@@ Coverage Diff @@
## 5.x #12899 +/- ##
=========================================
Coverage 63.13% 63.13%
Complexity 34446 34446
=========================================
Files 2266 2266
Lines 103051 103051
=========================================
Hits 65064 65064
Misses 37987 37987
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@kuzmany I am a bit unclear on how to test this PR. Could you please add more details to the test steps like where the screen is and possibly a screenshot as well? Thank you