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

Resolves the Milliseconds Conversion #604

Merged
merged 2 commits into from
Nov 6, 2023
Merged

Conversation

heyrandhir
Copy link
Contributor

@heyrandhir heyrandhir commented Nov 6, 2023

Issue Ticket Number:-

Backend changes

  • Yes
  • No

Frontend Changes

  • Yes
  • No

Database changes

  • Yes
  • No

Breaking changes (If your feature is breaking/missing something please mention pending tickets)

  • Yes
  • No

Deployment notes

None

Description

Eliminates the need for converting to milliseconds since the input was already in milliseconds.

Testing Stats:

image

image

@heyrandhir heyrandhir changed the title removes the conversion to ms as it was already in ms Resolves the Milliseconds Conversion Nov 6, 2023
Copy link

cloudflare-workers-and-pages bot commented Nov 6, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8304cd1
Status: ✅  Deploy successful!
Preview URL: https://fcc158db.dashboard-rds.pages.dev
Branch Preview URL: https://fix-resolve-util-fn.dashboard-rds.pages.dev

View logs

@heyrandhir heyrandhir self-assigned this Nov 6, 2023
@heyrandhir heyrandhir merged commit 9b52392 into develop Nov 6, 2023
5 checks passed
@heyrandhir heyrandhir deleted the fix/resolve-util-fn branch November 6, 2023 09:11
@heyrandhir heyrandhir mentioned this pull request Nov 6, 2023
12 tasks
heyrandhir added a commit that referenced this pull request Nov 6, 2023
* feature flag removal

* Fix Task Request Details Page (#602)

* fix oversize image, adds toasts and assignedTo

* adds modal on click of avatar and name

* test for modal

* fix failing test

* change cursor style to indicate clickable

* fix failing test

* fix the failing test condition

* add safety check for taskrequest which dont have users info

* resolve Sunny comment on the PR

* resolve comments from Bhavika

* fixes the event listener on the global window obj

* fixes failing test

* rename variable and remove commented code

* Resolves the Milliseconds Conversion (#604)

* removes the conversion to ms as it was already in ms

* modifies the timestamp as per the current format in fixtures

---------

Co-authored-by: meanwhile7 <[email protected]>
Co-authored-by: Vishal kamat <[email protected]>
Co-authored-by: Amit Prakash <[email protected]>
iamitprakash added a commit that referenced this pull request Nov 9, 2023
Dev to Main Sync (#605)

* feature flag removal

* Fix Task Request Details Page (#602)

* fix oversize image, adds toasts and assignedTo

* adds modal on click of avatar and name

* test for modal

* fix failing test

* change cursor style to indicate clickable

* fix failing test

* fix the failing test condition

* add safety check for taskrequest which dont have users info

* resolve Sunny comment on the PR

* resolve comments from Bhavika

* fixes the event listener on the global window obj

* fixes failing test

* rename variable and remove commented code

* Resolves the Milliseconds Conversion (#604)

* removes the conversion to ms as it was already in ms

* modifies the timestamp as per the current format in fixtures

---------

Co-authored-by: Randhir Kumar Singh <[email protected]>
Co-authored-by: meanwhile7 <[email protected]>
Co-authored-by: Vishal kamat <[email protected]>
Co-authored-by: Amit Prakash <[email protected]>
iamitprakash added a commit that referenced this pull request Nov 9, 2023
Dev to Main Sync (#605)

* feature flag removal

* Fix Task Request Details Page (#602)

* fix oversize image, adds toasts and assignedTo

* adds modal on click of avatar and name

* test for modal

* fix failing test

* change cursor style to indicate clickable

* fix failing test

* fix the failing test condition

* add safety check for taskrequest which dont have users info

* resolve Sunny comment on the PR

* resolve comments from Bhavika

* fixes the event listener on the global window obj

* fixes failing test

* rename variable and remove commented code

* Resolves the Milliseconds Conversion (#604)

* removes the conversion to ms as it was already in ms

* modifies the timestamp as per the current format in fixtures

---------

Co-authored-by: Randhir Kumar Singh <[email protected]>
Co-authored-by: meanwhile7 <[email protected]>
Co-authored-by: Vishal kamat <[email protected]>
Co-authored-by: Amit Prakash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants