-
Notifications
You must be signed in to change notification settings - Fork 14k
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
feat(viz-plugins): add date formatting to pivot-table #10637
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
* master: (43 commits) feat: Getting fancier with Storybook (apache#10647) fix: dedup groupby in viz.py while preserving order (apache#10633) feat: bump superset-ui for certified tag (apache#10650) feat: setup react page with submenu for datasources listview (apache#10642) feat: add certification to metrics (apache#10630) feat(viz-plugins): add date formatting to pivot-table (apache#10637) fix: controls scroll issue (apache#10644) feat: Allow tests files in /src (plus Label component tests) (apache#10634) fix: remove duplicated params and cache_timeout from list_columns; add viz_type to list_columns (apache#10643) chore: splitting button stories into separate stories (apache#10631) refactor: remove slice level label_colors from dashboard init load (apache#10603) feat: card view bulk select (apache#10607) style: Label styling/storybook touchups (apache#10627) fix: removing unsupported modal sizes (apache#10625) feat(datasource): remove deleted columns and update column type on metadata refresh (apache#10619) improve documentation for country maps (apache#10621) chore: npm audit fix as of 2020-08-15 (apache#10613) feat: dataset REST API for distinct values (apache#10595) chore: bump react-redux to 5.1.2, whittling console noise (apache#10602) fixing console error about bad html attribute (apache#10604) ... # Conflicts: # superset-frontend/src/explore/components/ExploreViewContainer.jsx # superset-frontend/src/views/App.tsx # superset/config.py
* feat: make pivot table dates formattable * Bump npm packages
* feat: make pivot table dates formattable * Bump npm packages
* feat: make pivot table dates formattable * Bump npm packages
* feat: make pivot table dates formattable * Bump npm packages
* feat: make pivot table dates formattable * Bump npm packages
* feat: make pivot table dates formattable * Bump npm packages
|
||
def _format_datetime(value: Any) -> Optional[str]: | ||
if isinstance(value, str): | ||
return f"__timestamp:{datetime_to_epoch(pd.Timestamp(value))}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@villebro it seems this only supports unix timestamp data type, not datetime
. E.g. column generated from SELECT now()
or SELECT date_trunc(TIMESTAMP '2020-01-01 00:00:00')
in PostgreSQL.
* feat: make pivot table dates formattable * Bump npm packages
SUMMARY
This PR adds the possibility to format dates in the
pivot-table
chart and also fixes some bugs in the control panel. Requires bumping@superset-ui/legacy-plugin-chart-pivot-table
to0.14.21
based on work in apache-superset/superset-ui#734 . Also bumps@superset-ui/legacy-preset-chart-nvd3
to0.14.21
to implement fix introduced in apache-superset/superset-ui#713 (ping @Mikedu1988 )AFTER
BEFORE
TEST PLAN
ADDITIONAL INFORMATION