mautic icon indicating copy to clipboard operation
mautic copied to clipboard

Report - fix date data display for local timezone

Open kuzmany opened this issue 2 years ago • 2 comments

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:

  1. Open this PR on Gitpod or pull down for testing locally (see docs on testing PRs here)
  2. Create custom field date for contact
  3. Create some contacts with values in that field
  4. Create report with the list of contacts with the date field in the column
  5. Just check If date field is displayed in report like before

kuzmany avatar Nov 16 '23 06:11 kuzmany

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

Impacted file tree graph

@@            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.

codecov[bot] avatar Dec 15 '23 08:12 codecov[bot]

@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

MojisolaaaO avatar Oct 04 '24 11:10 MojisolaaaO