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

fix: format table viz totals #2693

Merged
merged 1 commit into from Jul 13, 2022
Merged

fix: format table viz totals #2693

merged 1 commit into from Jul 13, 2022

Conversation

ZeRego
Copy link
Contributor

@ZeRego ZeRego commented Jul 13, 2022

Totals in the table viz weren't formatted like the values in the column.

Closes: #2695

Description:

Screenshot 2022-07-13 at 11 01 28

@ZeRego ZeRego added 🐛 bug Something isn't working as expected. This can be an error or an unexpected user experience 🖥 frontend This issue requires changes to the frontend labels Jul 13, 2022
@ZeRego ZeRego requested a review from a team July 13, 2022 10:02
@ZeRego ZeRego self-assigned this Jul 13, 2022
@ZeRego ZeRego added this to ⏳ In progress in Selected for Development via automation Jul 13, 2022
@owlas owlas requested a deployment to fix/format-table-viz-totals - jaffle_db PR #2693 July 13, 2022 10:03 — with Render Abandoned
@netlify
Copy link

netlify bot commented Jul 13, 2022

Deploy Preview for peaceful-bassi-cbf284 canceled.

Name Link
🔨 Latest commit 811d136
🔍 Latest deploy log https://app.netlify.com/sites/peaceful-bassi-cbf284/deploys/62cea9af952060000816b297

@12ian34
Copy link
Contributor

12ian34 commented Jul 13, 2022

Fixes #2695

@rephus
Copy link
Collaborator

rephus commented Jul 13, 2022

Code looks good, @12ian34 would you like to test this on render before we merge? otherwise I can do it . Makes sure it doesn't break anything else (test edge cases, different fields, etc) .

@ZeRego ZeRego removed this from ⏳ In progress in Selected for Development Jul 13, 2022
@ZeRego ZeRego removed 🐛 bug Something isn't working as expected. This can be an error or an unexpected user experience 🖥 frontend This issue requires changes to the frontend labels Jul 13, 2022
@ZeRego ZeRego force-pushed the fix/format-table-viz-totals branch from 308053e to 811d136 Compare July 13, 2022 11:17
@owlas owlas temporarily deployed to fix/format-table-viz-totals - lightdash PR #2693 July 13, 2022 11:17 — with Render Destroyed
@ZeRego ZeRego merged commit bcd81d7 into main Jul 13, 2022
@ZeRego ZeRego deleted the fix/format-table-viz-totals branch July 13, 2022 11:17
github-actions bot pushed a commit that referenced this pull request Jul 13, 2022
# [0.181.0](0.180.1...0.181.0) (2022-07-13)

### Bug Fixes

* edit table calculation ([#2696](#2696)) ([eb0871d](eb0871d))
* filter from table ([#2682](#2682)) ([c2e79bb](c2e79bb))
* format table viz totals ([#2693](#2693)) ([bcd81d7](bcd81d7))

### Features

* grouping dimensions and metrics ([#2680](#2680)) ([10b6740](10b6740))
@github-actions
Copy link

🎉 This PR is included in version 0.181.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Table chart totals don't apply same formatting as underlying data
4 participants