Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Change to fix datetime timezone issue #8625

Merged
merged 3 commits into from
Aug 30, 2023
Merged

Change to fix datetime timezone issue #8625

merged 3 commits into from
Aug 30, 2023

Conversation

hanzlamateen
Copy link
Member

@hanzlamateen hanzlamateen commented Aug 30, 2023

Summary

We are storing data in database in UTC timezone. Though while retrieving it, feathers was assuming it to be in local time. Therefore added resolvers to convert the time to UTC on server before returning it to client.

References

closes #insert number here

Checklist

  • If this PR is still a WIP, convert to a draft
  • When this PR is ready, mark it as "Ready for review"
  • ensure all checks pass
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewer

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

@hanzlamateen hanzlamateen marked this pull request as ready for review August 30, 2023 09:56
@hanzlamateen hanzlamateen merged commit 72d46ed into dev Aug 30, 2023
8 checks passed
@hanzlamateen hanzlamateen deleted the datetime-fix branch August 30, 2023 10:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant