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.

Sign up for GitHub

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

Jump to bottom

Distinct list results in raw date_time format when applied on Datetime with timezone columns. #2966

Closed
Tracked by #2962
Aritra8438 opened this issue Jun 22, 2023 · 2 comments · Fixed by #3325
Closed
Tracked by #2962

Distinct list results in raw date_time format when applied on Datetime with timezone columns. #2966

Aritra8438 opened this issue Jun 22, 2023 · 2 comments · Fixed by #3325
Assignees
@dikwickley
Labels
ready Ready for implementation type: bug Something isn't working work: frontend Related to frontend code in the mathesar_ui directory
Milestone
Backlog

Comments

@Aritra8438
Copy link
Member

Description

If Distinct List transformation is applied on Datetime with timezone columns, it generates pills of raw datetime format.

By raw datetime format, I meant this format: 2023-06-08T06:30:00Z.
We should calculate the time with timezone before rendering it.

Expected behavior

The format of the pills should adhere to the cell representaion of Datetime with timezone type in table.
That looks like: 06/22/2023 12:00

Screenshot

image
@Aritra8438 Aritra8438 added type: bug Something isn't working status: triage labels Jun 22, 2023
@Aritra8438 Aritra8438 mentioned this issue Jun 22, 2023
Closed
@Aritra8438 Aritra8438 self-assigned this Jun 22, 2023
@rajatvijay rajatvijay added this to the Backlog milestone Jul 5, 2023
@rajatvijay rajatvijay added work: frontend Related to frontend code in the mathesar_ui directory status: draft labels Jul 11, 2023
@seancolsen seancolsen added ready Ready for implementation and removed status: draft labels Nov 1, 2023
@seancolsen
Copy link
Contributor

Unassigning due to inactivity

@dikwickley dikwickley mentioned this issue Nov 26, 2023
Merged
7 tasks
@dikwickley
Copy link

@seancolsen This issue was still there so submitted a PR to fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dikwickley dikwickley

Labels
ready Ready for implementation type: bug Something isn't working work: frontend Related to frontend code in the mathesar_ui directory
Projects
No open projects
[NO LONGER USED] Mathesar work tracker
Status: Done
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

Fixed formatting strings for timestampsWithTZ dikwickley/mathesar
4 participants
@seancolsen @rajatvijay @dikwickley @Aritra8438