user token expired warning #7040

NamanMahor · 2025-04-02T07:17:33Z

raise new PR #7124

Checklist:

Covered by tests
Ran it and it works as intended
Reviewed the diff before requesting a review
Checked for unhandled edge cases
Linked the issues it closes
Checked if the docs need to be updated
Intend to cherry-pick into the release branch
I'm proud of this work!

cli/pkg/dotrill/dotrill.go

cli/cmd/root.go

cli/cmd/sudo/user/assume.go

cli/cmd/root.go

cli/cmd/sudo/user/assume.go

cli/cmd/sudo/user/unassume.go

cli/pkg/dotrill/dotrill.go

Co-authored-by: Benjamin Egelund-Müller <[email protected]>

#7039) * fix: watermark field is converted to timestamp when using transform_sql * justify blank import * remove json file dump completely

* tweak: canvas chart grid lines * Update canvas reference * solid bars

* Move merging of state to a single place * Integrate with rill-dev explore * Remove all references of DashboardURLStateSync * Keep the old merge behaviour for now * Update homebookmark undefined => null

* athena integration test * update env variable

* Add init time filter tests * chore: add e2e canvas filters test

* cleanup transporters interface * fix unit tests

* bigquery to clickhouse connector * fix gitignore * fix lint * fix lint * fix lint * fix lint * review comments

* default reorderMeasuresDimensions to true * remove reorederMeasuresDimensions flag * lint * aria for test, dimension and measure test fix * lint * test fix * test fix * test fix * data-testid, test fix * try * hide sum of bid price, test fix * test fix * clean up * test fix * fixme

* Honor order of dims and measures from url param * Add a test * Remove references of the old Set * PR comments * Add back toggleAllXX * Fix merge * set to arry in dashboard state manager loaders --------- Co-authored-by: Cyrus Goh <[email protected]>

* rename, remove unused * dimension gradients, optional by count * rename for readability, fix dimension and measure cell gradients * clean up * offline sync, revert title * lint

…was not DuckDB (#7048) * Better logic to show dialog step 2 * Nit: tweak padding

* Simplify session storage to work off of url params * Unify getter methods

* Test fixtures for `admin` and `cli` * CLI fixtures * Self review * Merge * Merge * QA * Gen docs * Fixitesto? * Fix compiler err

…ther resources (#7046) * Use plural (as we do for other resources in this section) * Use plural in-app too * Update more links * Update one more link * Update more links

* Improve unstable e2e cloud setup * Ensure duckdb extensions * Cleanup

…'s sorted (#7067) * fetch leaderboard measures, perf fix * rename `leaderboardMeasureName` to `leaderboardSortByMeasureName` * Two more renames * Fix import --------- Co-authored-by: Cyrus Goh <[email protected]>

* adding pinger for druid driver

* fix: remove none option from timezone selector * test: update visual explore editing test

* init commit * type and api fixes * chore: remove unused import * fix: add back gc param * fix: type signature * fix: type and api updates * fix: remove logs * refac: remove refetchOnMount override * fix: remove log * fix: type issue * fix: type fixes * fix: attempt to fix breadcrumb test * Two bugfixes * fix: leaderboard loading state, deprecate useModelHasTimeSeries * fix: change implementation to use isPending * Re-generate admin client with `infiniteQueryParams` * Add `getNextPageParam` functions to admin infinite queries * chore: update orval to 7.8, regen clients * fix: type fix * fix: test retries --------- Co-authored-by: Eric P Green <[email protected]>

* decreasing first when toggling new measure * fix toggling diff sort type, descending * clean up * readable

* Clean up runtime protos * Remove deprecated metrics view fields (#7074) * Remove deprecated metrics view fields * Fix checks * Remove `V2` suffix from `MetricsView` type (#7075) * Remove bucket planner const

#7080)

… to Explore views (#7084)

* Create 0.59.md * first rough pass * docs: deprecation notice of password auth for sf * docs: release notes * docs: adding gif * update --------- Co-authored-by: Alexander Thor <[email protected]>

* chore: remove .css from prettierignore, consolidate vega css * chore: remove commented out styles

* init commit * feat: component initialization * fix: type fix and unused imports * chore: remove unused imports * feat: responsive spec stores, component property * chore: cleanup * fix: test fixes * fix: type fix * fix: remove debug element * feat: consolidate table components, update layout method for optimized rendering * fix: invalid import * chore: cleanup * chore: even more cleanup * chore: remove log * chore: remove comment * chore: remove log * chore: remove logs * fix: types, cleanup * chore: prettier * fix: type issues * fix: range display kpi styles * chore: add back comments * chore: add back TableTemplateT * fix: import * fix: time comparison conditional * chore: add comments * fix: types * fix: switching table type * chore: remove debug additions * fix: filter test * revert package-lock changes * fix: set initial filter state

* Add "Create new model" to model context menu * Fix autocomplete of source tables * Patch type * Fix test * Little test cleanup

* fix: allows show custom of legacy dashboards * fix: added comment for clarity

…operty (#7087) * Remove `exploreState.pivot.active` property (in favor of `exploreState.activePage`) * Add tests to delete bookmarks * Fix logic for filter-only bookmarks * Fix unit test * Review * Fix merge

* druid version check * Update runtime/drivers/druid/druid.go Co-authored-by: Benjamin Egelund-Müller <[email protected]> * Update runtime/drivers/druid/druid.go Co-authored-by: Copilot <[email protected]> * review --------- Co-authored-by: Benjamin Egelund-Müller <[email protected]> Co-authored-by: Copilot <[email protected]>

* Detach attached db and remove views from databases holding duckdb tables * always close connection * exclude motherduck databases * use db and not conn * use the same connection

* fix: remove event listeners from expandable cells * fix: update test selector

NamanMahor · 2025-04-10T04:43:39Z

closing it as rebased it :(

user_token_expired_warning

a1b8ed7

NamanMahor requested review from begelundmuller and k-anshul April 2, 2025 07:17

fix lint

c83ca04

NamanMahor self-assigned this Apr 2, 2025

NamanMahor added 2 commits April 2, 2025 18:26

unassume user if already assumed

e9eabd2

unassume user if already assumed

10952da

k-anshul reviewed Apr 3, 2025

View reviewed changes

cli/pkg/dotrill/dotrill.go Outdated Show resolved Hide resolved

cli/cmd/root.go Outdated Show resolved Hide resolved

k-anshul reviewed Apr 3, 2025

View reviewed changes

cli/cmd/sudo/user/assume.go Outdated Show resolved Hide resolved

review comment

0bf451d

NamanMahor requested a review from k-anshul April 9, 2025 10:44

begelundmuller requested changes Apr 9, 2025

View reviewed changes

NamanMahor and others added 18 commits April 10, 2025 09:36

review comments

dabcf39

Update cli/cmd/sudo/user/assume.go

cd1b766

Co-authored-by: Benjamin Egelund-Müller <[email protected]>

fix: watermark field is converted to string when using transform_sql (

44e6048

#7039) * fix: watermark field is converted to timestamp when using transform_sql * justify blank import * remove json file dump completely

tweak: canvas chart grid lines (#7011)

3425011

* tweak: canvas chart grid lines * Update canvas reference * solid bars

fix: Move merging of state to a single place (#7035)

06d1a4f

* Move merging of state to a single place * Integrate with rill-dev explore * Remove all references of DashboardURLStateSync * Keep the old merge behaviour for now * Update homebookmark undefined => null

athena integration test (#6969)

d691398

* athena integration test * update env variable

chore: add canvas filter tests (#7025)

277ae30

* Add init time filter tests * chore: add e2e canvas filters test

cleanup transporters interface (#7045)

fd48aeb

* cleanup transporters interface * fix unit tests

bigquery to clickhouse connector (#7012)

dcd8f0c

* bigquery to clickhouse connector * fix gitignore * fix lint * fix lint * fix lint * fix lint * review comments

Fix leaderboard bg gradients (#7037)

d5a747e

* rename, remove unused * dimension gradients, optional by count * rename for readability, fix dimension and measure cell gradients * clean up * offline sync, revert title * lint

update leaderboard measure when hiding measures (#7052)

a39f606

fix (Add Data): OLAP forms were hidden when your default OLAP engine …

35e1223

…was not DuckDB (#7048) * Better logic to show dialog step 2 * Nit: tweak padding

Update install script to support both shasum and sha256sum (#7056)

465b096

Simplify session storage to work off of url params (#7041)

11c6a9d

* Simplify session storage to work off of url params * Unify getter methods

Test fixtures for admin and cli (#6982)

1b0ce8d

* Test fixtures for `admin` and `cli` * CLI fixtures * Self review * Merge * Merge * QA * Gen docs * Fixitesto? * Fix compiler err

fix (docs): MetricsView reference page should use plural like the o…

b1753ab

…ther resources (#7046) * Use plural (as we do for other resources in this section) * Use plural in-app too * Update more links * Update one more link * Update more links

AdityaHegde and others added 27 commits April 10, 2025 10:10

fix: unstable e2e cloud setup (#7053)

947de9f

* Improve unstable e2e cloud setup * Ensure duckdb extensions * Cleanup

adding pinger for druid driver (#7050)

2fbea2a

* adding pinger for druid driver

fix: hide tab indicator (#7059)

a461bb1

fix: editor focus stealing (#7058)

0a8c065

fix: remove none option from timezone selector (#7060)

97f719f

* fix: remove none option from timezone selector * test: update visual explore editing test

Decreasing first when toggling new measure (#7066)

8b99aee

* decreasing first when toggling new measure * fix toggling diff sort type, descending * clean up * readable

ability to deselect hidden item (#7068)

fb36227

chore: upgrade vitest (#7077)

85ee4eb

change_install_and_load_order (#7072)

4ea3b66

Clean up runtime protos (#7073)

c909489

* Clean up runtime protos * Remove deprecated metrics view fields (#7074) * Remove deprecated metrics view fields * Fix checks * Remove `V2` suffix from `MetricsView` type (#7075) * Remove bucket planner const

Enable bigquery connector to query google sheet backed external tables (

6987439

#7080)

chore: regen package lock (#7079)

2841262

refactor (state management): use explicit activePage state to route…

986b858

… to Explore views (#7084)

docs: 0.59 release notes (#7076)

b95ee16

* Create 0.59.md * first rough pass * docs: deprecation notice of password auth for sf * docs: release notes * docs: adding gif * update --------- Co-authored-by: Alexander Thor <[email protected]>

Add tests to delete bookmarks (#7091)

61e2a5d

return ctx err for druid driver (#7093)

05cf6fc

chore: remove .css from prettierignore, consolidate vega css (#7078)

99e6d2e

* chore: remove .css from prettierignore, consolidate vega css * chore: remove commented out styles

fix (Sources): Handle Sources-as-Models in UI (#7088)

85e0bea

* Add "Create new model" to model context menu * Fix autocomplete of source tables * Patch type * Fix test * Little test cleanup

fix: allows show custom of legacy dashboards (#7100)

e739d11

* fix: allows show custom of legacy dashboards * fix: added comment for clarity

refactor (state management): Deprecate exploreState.pivot.active pr…

7ac17bc

…operty (#7087) * Remove `exploreState.pivot.active` property (in favor of `exploreState.activePage`) * Add tests to delete bookmarks * Fix logic for filter-only bookmarks * Fix unit test * Review * Fix merge

Update pinot go client (#7096)

3e72b93

Cleanup attached db (#7071)

ad7914b

* Detach attached db and remove views from databases holding duckdb tables * always close connection * exclude motherduck databases * use db and not conn * use the same connection

fix: remove event listeners from expandable cells (#7108)

6dc2002

* fix: remove event listeners from expandable cells * fix: update test selector

NamanMahor closed this Apr 10, 2025

NamanMahor deleted the user_token_expired_warning branch April 10, 2025 07:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

user token expired warning #7040

user token expired warning #7040

NamanMahor commented Apr 2, 2025 •

edited

Loading

NamanMahor commented Apr 10, 2025

user token expired warning #7040

user token expired warning #7040

Conversation

NamanMahor commented Apr 2, 2025 • edited Loading

NamanMahor commented Apr 10, 2025

NamanMahor commented Apr 2, 2025 •

edited

Loading