Skip to content

[O2B-1083] Fix a bug in LHC overview pages, now displaying missing dates as '-' instead of the default epoch time#1298

Merged
graduta merged 5 commits into
mainfrom
SandroBoelsz/O2B-1083/fix-empty-date-to-start-epoch-when-null-bug
Dec 14, 2023
Merged

[O2B-1083] Fix a bug in LHC overview pages, now displaying missing dates as '-' instead of the default epoch time#1298
graduta merged 5 commits into
mainfrom
SandroBoelsz/O2B-1083/fix-empty-date-to-start-epoch-when-null-bug

Conversation

@SandroBoelsz

@SandroBoelsz SandroBoelsz commented Dec 13, 2023

Copy link
Copy Markdown
Contributor

I have a JIRA ticket

  • branch and/or PR name(s) include(s) JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Notable changes for users:

  • Missing dates on the LHC overview pages are now shown to the user as '-' instead of 01/01/1970 01:00:00

Notable changes for developers:

  • The LHCFillAdapter now verifies whether a date is null/undefined and refrains from using new Date() when encountering missing values

Changes made to the database:

  • None

@SandroBoelsz SandroBoelsz self-assigned this Dec 13, 2023
@SandroBoelsz SandroBoelsz added bug Something isn't working backend labels Dec 13, 2023
@codecov

codecov Bot commented Dec 13, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6359923) 45.58% compared to head (04439fc) 45.59%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1298   +/-   ##
=======================================
  Coverage   45.58%   45.59%           
=======================================
  Files         699      699           
  Lines       11090    11092    +2     
  Branches     2023     2025    +2     
=======================================
+ Hits         5055     5057    +2     
  Misses       6035     6035           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread lib/database/adapters/LhcFillAdapter.js Outdated
@graduta
graduta merged commit 96f7ca3 into main Dec 14, 2023
@graduta
graduta deleted the SandroBoelsz/O2B-1083/fix-empty-date-to-start-epoch-when-null-bug branch December 14, 2023 12:16
CollinPoetoehena added a commit that referenced this pull request Dec 14, 2023
* [O2B-1077] Keep scroll in popover after re-rendering (#1300)

* [O2B-1066]: Added extra check for empty for log environment filter (#1296)

* [O2B-1066]: Added extra check for empty value after initialization

* [O2B-1083] Fix a bug in LHC overview pages, now displaying missing dates as '-' instead of the default epoch time (#1298)

* [1084] add getOrCreate function to UserService (#1294)

* feat: add getorcreate function in UserService

* test: add tests for get or create user

* refactor: create or update user service

* test: update tests with new create or update service

* fix: file paths create or update user

* refactor: delete unused old usecase for users

* fix: update user test broken

* refactor: add update chech + code improvement

* test: add test case for not updating user

* refactor: implement comments on PR

* refactor: move one time used code in create user

---------

Co-authored-by: Martin Boulais <31805063+martinboulais@users.noreply.github.com>
Co-authored-by: mennokonijn <99171568+mennokonijn@users.noreply.github.com>
Co-authored-by: SandroBoelsz <97670955+SandroBoelsz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend bug Something isn't working

Development

Successfully merging this pull request may close these issues.

3 participants