[I] After clearning selceted values on Dashboard filters deosn't show all value [superset]

2024-04-11 Thread via GitHub
Davidkramer1999 opened a new issue, #28004: URL: https://github.com/apache/superset/issues/28004 ### Bug description So if you have filter on dashboard. Select the value whihc is not present in dropdown. Clear the value and after opening dropdown it won't show all values as initially

Re: [I] [SIP-123] Proposal replacement of data table components with ag-grid [superset]

2024-04-11 Thread via GitHub
aikawa-ohno commented on issue #27645: URL: https://github.com/apache/superset/issues/27645#issuecomment-2050950783 Will #26364 column filter like the one in a be implemented? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [I] [DRAFT - WiP][SIP] Proposal for Enhanced Data Access Permissions [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on issue #28002: URL: https://github.com/apache/superset/issues/28002#issuecomment-2050935920 ![image](https://github.com/apache/superset/assets/487433/41f9f3de-ef20-4abe-9ab3-adf2ca2ef1ad) -- This is an automated message from the Apache Git Service. To respond

Re: [I] [DRAFT - WiP][SIP] Proposal for Enhanced Data Access Permissions [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on issue #28002: URL: https://github.com/apache/superset/issues/28002#issuecomment-2050933436 ![image](https://github.com/apache/superset/assets/487433/818c6642-45a3-46bd-b6a2-7787c310412a) -- This is an automated message from the Apache Git Service. To respond

Re: [PR] chore(ASF): adds DOAP file and bumping apache-rat [superset]

2024-04-11 Thread via GitHub
rusackas commented on code in PR #27978: URL: https://github.com/apache/superset/pull/27978#discussion_r1561986724 ## ASF/doap_Superset.rdf: ## @@ -0,0 +1,63 @@ + + +http://usefulinc.com/ns/doap#"; + xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"; + xmln

Re: [I] [DRAFT - WiP][SIP] Proposal for Enhanced Data Access Permissions [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on issue #28002: URL: https://github.com/apache/superset/issues/28002#issuecomment-2050929306 ```plantuml @startuml ' hide the spot ' hide circle ' avoid problems with angled crows feet skinparam linetype ortho 'left to right direction

[I] [DRAFT - WiP][SIP] Proposal for Enhanced Data Access Permissions [superset]

2024-04-11 Thread via GitHub
mistercrunch opened a new issue, #28002: URL: https://github.com/apache/superset/issues/28002 ### [SIP] Proposal for Enhanced Data Access Permissions Motivation Superset's current permission model relies on Flask-AppBuilder (FAB) for both app-level and data access permi

Re: [PR] feat(accessibility): add tabbing to chart menu in dashboard [superset]

2024-04-11 Thread via GitHub
eschutho commented on code in PR #26138: URL: https://github.com/apache/superset/pull/26138#discussion_r1561871095 ## superset-frontend/src/dashboard/types.ts: ## @@ -237,3 +237,32 @@ export type Slice = { owners: { id: number }[]; created_by: { id: number }; }; + +export

[I] BYTEA column value decoding breaks preview and select. "'utf-8' codec can't decode byte 0xe6 in position 0: invalid continuation byte" [superset]

2024-04-11 Thread via GitHub
dvdotsenko opened a new issue, #28001: URL: https://github.com/apache/superset/issues/28001 ### Bug description Preview and select functionalities break in SuperSet UI on columns of BYTEA type. ``` Database error Failed to execute query '15' - 'SELECT addressid,

Re: [PR] chore(ci): make pre-commit step faster by skipping superset install [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on code in PR #27979: URL: https://github.com/apache/superset/pull/27979#discussion_r1561851846 ## .github/workflows/pre-commit.yml: ## @@ -35,6 +37,7 @@ jobs: brew install norwoodj/tap/helm-docs - name: pre-commit run: | +

[PR] feat: action-specific py deps pinning [superset]

2024-04-11 Thread via GitHub
mistercrunch opened a new pull request, #28000: URL: https://github.com/apache/superset/pull/28000 Had this idea around addressing the comment here: https://github.com/apache/superset/pull/27979#pullrequestreview-1994777679 , introducing action-specific pinning. I tested that this wo

Re: [PR] fix(time_offset): improved LIMIT-handling in advanced analytics [superset]

2024-04-11 Thread via GitHub
Antonio-RiveroMartnez merged PR #27934: URL: https://github.com/apache/superset/pull/27934 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications

Re: [PR] fix(time_offset): improved LIMIT-handling in advanced analytics [superset]

2024-04-11 Thread via GitHub
github-actions[bot] commented on PR #27934: URL: https://github.com/apache/superset/pull/27934#issuecomment-2050684468 Ephemeral environment shutdown and build artifacts deleted. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[PR] Revert "chore(ci): make pre-commit step faster by skipping superset install" [superset]

2024-04-11 Thread via GitHub
mistercrunch opened a new pull request, #27999: URL: https://github.com/apache/superset/pull/27999 Reverts apache/superset#27979 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

Re: [PR] chore(ci): make pre-commit step faster by skipping superset install [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on code in PR #27979: URL: https://github.com/apache/superset/pull/27979#discussion_r1561821563 ## .github/workflows/pre-commit.yml: ## @@ -35,6 +37,7 @@ jobs: brew install norwoodj/tap/helm-docs - name: pre-commit run: | +

Re: [PR] chore(ci): make pre-commit step faster by skipping superset install [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on code in PR #27979: URL: https://github.com/apache/superset/pull/27979#discussion_r1561821992 ## .github/workflows/pre-commit.yml: ## @@ -35,6 +37,7 @@ jobs: brew install norwoodj/tap/helm-docs - name: pre-commit run: | +

Re: [I] Error message when trying to query mongodb databse using superset meta database connection [superset]

2024-04-11 Thread via GitHub
rusackas commented on issue #27989: URL: https://github.com/apache/superset/issues/27989#issuecomment-2050660610 Hopefully @betodealmeida can help a bit. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] chore: set up git lfs for larger files [superset]

2024-04-11 Thread via GitHub
mistercrunch closed pull request #27923: chore: set up git lfs for larger files URL: https://github.com/apache/superset/pull/27923 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment

[PR] docs: move mp4 video to superset-site/tree/lfs [superset]

2024-04-11 Thread via GitHub
mistercrunch opened a new pull request, #27998: URL: https://github.com/apache/superset/pull/27998 This big video file doesn't belong in the main repo! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

Re: [PR] chore: set up git lfs for larger files [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on PR #27923: URL: https://github.com/apache/superset/pull/27923#issuecomment-2050659088 Ok this is a bit disruptive, decided to go with this approach instead -> https://github.com/apache/superset/pull/27998 -- This is an automated message from the Apache Git Servic

Re: [PR] feat: Slack Avatar integration [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on code in PR #27849: URL: https://github.com/apache/superset/pull/27849#discussion_r1561788473 ## superset/config.py: ## @@ -1328,6 +1328,8 @@ def EMAIL_HEADER_MUTATOR( # pylint: disable=invalid-name,unused-argument # Slack API token for the superset r

Re: [PR] chore(ci): make pre-commit step faster by skipping superset install [superset]

2024-04-11 Thread via GitHub
villebro commented on code in PR #27979: URL: https://github.com/apache/superset/pull/27979#discussion_r1561785640 ## .github/workflows/pre-commit.yml: ## @@ -35,6 +37,7 @@ jobs: brew install norwoodj/tap/helm-docs - name: pre-commit run: | +

Re: [I] Redacted [superset]

2024-04-11 Thread via GitHub
rusackas commented on issue #27987: URL: https://github.com/apache/superset/issues/27987#issuecomment-2050596618 Thank you for caring enough to spot/report it... just closing this so we can tackle it further from public view. -- This is an automated message from the Apache Git Service. To

Re: [I] Redacted [superset]

2024-04-11 Thread via GitHub
rusackas closed issue #27987: Redacted URL: https://github.com/apache/superset/issues/27987 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notification

Re: [I] Redacted [superset]

2024-04-11 Thread via GitHub
rusackas commented on issue #27987: URL: https://github.com/apache/superset/issues/27987#issuecomment-2050595725 I've removed the content of this issue. Please send this or any security concerns to secur...@superset.apache.org so we can assess/track/fix the issue in private, with you in the

Re: [PR] chore: set up git lfs for larger files [superset]

2024-04-11 Thread via GitHub
rusackas commented on PR #27923: URL: https://github.com/apache/superset/pull/27923#issuecomment-2050509211 Just noticed the geojson files are now moved to git-lfs, that means you need to have/use git-lfs to run Superset with maps now. This really ought to get covered in all the relevant po

Re: [PR] chore: set up git lfs for larger files [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on code in PR #27923: URL: https://github.com/apache/superset/pull/27923#discussion_r1561600883 ## .gitattributes: ## @@ -1,18 +1,8 @@ docker/**/*.sh text eol=lf *.mp4 filter=lfs diff=lfs merge=lfs -text -docs/static/img/hero-screenshot.jpg filter=lfs di

[PR] fix(ListView): avoid firing event on change of selection [superset]

2024-04-11 Thread via GitHub
justinpark opened a new pull request, #27997: URL: https://github.com/apache/superset/pull/27997 ### SUMMARY When you paste a value in the list view filter, the input field automatically submits the input value right away. This happens because the `onSelect` event is triggered when the

Re: [PR] feat: Slack Avatar integration [superset]

2024-04-11 Thread via GitHub
michael-s-molina commented on code in PR #27849: URL: https://github.com/apache/superset/pull/27849#discussion_r1561585283 ## superset/config.py: ## @@ -1328,6 +1328,8 @@ def EMAIL_HEADER_MUTATOR( # pylint: disable=invalid-name,unused-argument # Slack API token for the supers

Re: [PR] feat(explore): Color scheme groups, new color schemes [superset]

2024-04-11 Thread via GitHub
michael-s-molina commented on PR #27995: URL: https://github.com/apache/superset/pull/27995#issuecomment-2050403617 Maybe another question would be, does it really make a difference separating color pallets by featured / others / etc? If so, another possible alternative would be to render t

Re: [PR] feat(explore): Color scheme groups, new color schemes [superset]

2024-04-11 Thread via GitHub
michael-s-molina commented on PR #27995: URL: https://github.com/apache/superset/pull/27995#issuecomment-2050385763 @kgabryje I have some questions about how this behavior will work with some of the requirements/features of the Select component: - Selected items are always sorted first. H

Re: [PR] chore: set up git lfs for larger files [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on PR #27923: URL: https://github.com/apache/superset/pull/27923#issuecomment-2050374617 oh wow! - shallow clone is 0m20.222s VS 1m4.067s (3x faster) - size of `.git` 35M VS 161M (21% of the space) GPT-generated benchmark script ```bash #!/bin/bash

[PR] fix: Incorrect onChange value when an unloaded value is pasted into AsyncSelect [superset]

2024-04-11 Thread via GitHub
michael-s-molina opened a new pull request, #27996: URL: https://github.com/apache/superset/pull/27996 ### SUMMARY Fixes a bug where the `onChange` event was being fired with an incorrect value when pasting a text that corresponds to an existing option in the database, which was not load

Re: [PR] fix(drillby): Enable DrillBy in charts w/o filters (dimensions) [superset]

2024-04-11 Thread via GitHub
sowo commented on PR #27941: URL: https://github.com/apache/superset/pull/27941#issuecomment-2050334883 @kgabryje initially, I used the X axis value plus the selected dimension for the drill by chart. But this created a time series chart with just one bar broken down by the selected dimensi

Re: [PR] fix(docs): add missing code formatting, fix broken link [superset]

2024-04-11 Thread via GitHub
sfirke merged PR #27992: URL: https://github.com/apache/superset/pull/27992 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@su

Re: [PR] feat: Slack Avatar integration [superset]

2024-04-11 Thread via GitHub
rusackas commented on code in PR #27849: URL: https://github.com/apache/superset/pull/27849#discussion_r1561503023 ## superset/config.py: ## @@ -1453,6 +1455,7 @@ def EMAIL_HEADER_MUTATOR( # pylint: disable=invalid-name,unused-argument "data:", "https

Re: [PR] feat: Slack Avatar integration [superset]

2024-04-11 Thread via GitHub
rusackas commented on code in PR #27849: URL: https://github.com/apache/superset/pull/27849#discussion_r1561501550 ## superset/config.py: ## @@ -1328,6 +1328,8 @@ def EMAIL_HEADER_MUTATOR( # pylint: disable=invalid-name,unused-argument # Slack API token for the superset repor

Re: [PR] fix(docs): add missing code formatting, fix broken link [superset]

2024-04-11 Thread via GitHub
rusackas commented on PR #27992: URL: https://github.com/apache/superset/pull/27992#issuecomment-2050309336 Don't know what's up with all these queued github actions that never seem to spring to live. Closing/reopening to trigger stuff... -- This is an automated message from the Apache Gi

Re: [PR] fix(drillby): Enable DrillBy in charts w/o filters (dimensions) [superset]

2024-04-11 Thread via GitHub
rusackas merged PR #27941: URL: https://github.com/apache/superset/pull/27941 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@

Re: [PR] fix(docs): add missing code formatting, fix broken link [superset]

2024-04-11 Thread via GitHub
rusackas closed pull request #27992: fix(docs): add missing code formatting, fix broken link URL: https://github.com/apache/superset/pull/27992 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the sp

Re: [I] Drill by is disabled for Timeseries and Mixed Timeseries charts without dimensions [superset]

2024-04-11 Thread via GitHub
rusackas closed issue #27940: Drill by is disabled for Timeseries and Mixed Timeseries charts without dimensions URL: https://github.com/apache/superset/issues/27940 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] feat: add option to disable rendering of html in sql lab and table chart [superset]

2024-04-11 Thread via GitHub
rusackas merged PR #27969: URL: https://github.com/apache/superset/pull/27969 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@

Re: [I] Installing `superset-frontend` as a package throws an error which links to `@superset-ui/plugin-chart-period-over-period-kpi` dependency [superset]

2024-04-11 Thread via GitHub
michael-s-molina closed issue #27993: Installing `superset-frontend` as a package throws an error which links to `@superset-ui/plugin-chart-period-over-period-kpi` dependency URL: https://github.com/apache/superset/issues/27993 -- This is an automated message from the Apache Git Service. To

Re: [PR] fix(superset-frontend): remove unused `@superset-ui/plugin-chart-period-over-period-kpi` package [superset]

2024-04-11 Thread via GitHub
michael-s-molina merged PR #27994: URL: https://github.com/apache/superset/pull/27994 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsu

Re: [PR] chore(superset-frontend): remove unused `@superset-ui/plugin-chart-period-over-period-kpi` package [superset]

2024-04-11 Thread via GitHub
corocoto commented on PR #27994: URL: https://github.com/apache/superset/pull/27994#issuecomment-2050270567 @michael-s-molina, you are welcome! And thanks for your approve. But can you do it again? Because I changed the PR's title, and the `link-check` workflow waited an approval from a

Re: [PR] feat: add option to disable rendering of html in sql lab and table chart [superset]

2024-04-11 Thread via GitHub
justinpark commented on code in PR #27969: URL: https://github.com/apache/superset/pull/27969#discussion_r1561446661 ## superset-frontend/src/SqlLab/components/ResultSet/index.tsx: ## @@ -579,6 +580,10 @@ const ResultSet = ({ const expandedColumns = results.expanded_colum

Re: [PR] fix(drillby): Enable DrillBy in charts w/o filters (dimensions) [superset]

2024-04-11 Thread via GitHub
github-actions[bot] commented on PR #27941: URL: https://github.com/apache/superset/pull/27941#issuecomment-2050228179 Ephemeral environment shutdown and build artifacts deleted. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] fix(drillby): Enable DrillBy in charts w/o filters (dimensions) [superset]

2024-04-11 Thread via GitHub
rusackas closed pull request #27941: fix(drillby): Enable DrillBy in charts w/o filters (dimensions) URL: https://github.com/apache/superset/pull/27941 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go t

Re: [PR] fix(drillby): Enable DrillBy in charts w/o filters (dimensions) [superset]

2024-04-11 Thread via GitHub
rusackas commented on PR #27941: URL: https://github.com/apache/superset/pull/27941#issuecomment-2050227229 Hmm... closing/reopening to see if that kicks CI into gear. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] feat(alert report tabs): adding feature flag [superset]

2024-04-11 Thread via GitHub
rusackas merged PR #27773: URL: https://github.com/apache/superset/pull/27773 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@

Re: [PR] chore(ASF): adds DOAP file [superset]

2024-04-11 Thread via GitHub
rusackas commented on code in PR #27978: URL: https://github.com/apache/superset/pull/27978#discussion_r1561384632 ## ASF/doap_Superset.rdf: ## @@ -0,0 +1,63 @@ + + +http://usefulinc.com/ns/doap#"; + xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"; + xmln

Re: [PR] chore: set up git lfs for larger files [superset]

2024-04-11 Thread via GitHub
rusackas commented on PR #27923: URL: https://github.com/apache/superset/pull/27923#issuecomment-2050188256 > 100% seamless for anyone who has git-lfs installed This is the part that worries me... I assume most people don't? -- This is an automated message from the Apache Git Service.

Re: [PR] chore: set up git lfs for larger files [superset]

2024-04-11 Thread via GitHub
rusackas commented on code in PR #27923: URL: https://github.com/apache/superset/pull/27923#discussion_r1561381655 ## .gitattributes: ## @@ -1,18 +1,8 @@ docker/**/*.sh text eol=lf *.mp4 filter=lfs diff=lfs merge=lfs -text -docs/static/img/hero-screenshot.jpg filter=lfs diff=l

Re: [PR] chore: set up git lfs for larger files [superset]

2024-04-11 Thread via GitHub
rusackas commented on PR #27923: URL: https://github.com/apache/superset/pull/27923#issuecomment-2050183463 The only thing I'd ask for here is updates to the docs, particularly around testing/contrinbuting to the docs. If the process to check out and run the docs with images/video is differ

Re: [PR] chore(ci): make pre-commit step faster by skipping superset install [superset]

2024-04-11 Thread via GitHub
john-bodley commented on code in PR #27979: URL: https://github.com/apache/superset/pull/27979#discussion_r1561366109 ## .github/workflows/pre-commit.yml: ## @@ -35,6 +37,7 @@ jobs: brew install norwoodj/tap/helm-docs - name: pre-commit run: | +

Re: [PR] chore(ASF): adds DOAP file [superset]

2024-04-11 Thread via GitHub
michael-s-molina commented on code in PR #27978: URL: https://github.com/apache/superset/pull/27978#discussion_r1561358327 ## ASF/doap_Superset.rdf: ## @@ -0,0 +1,63 @@ + + +http://usefulinc.com/ns/doap#"; + xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"; +

Re: [PR] feat(alert report tabs): adding feature flag [superset]

2024-04-11 Thread via GitHub
fisjac closed pull request #27773: feat(alert report tabs): adding feature flag URL: https://github.com/apache/superset/pull/27773 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment

Re: [PR] chore: remove no-op.yml as it's not needed anymore [superset]

2024-04-11 Thread via GitHub
mistercrunch merged PR #27980: URL: https://github.com/apache/superset/pull/27980 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr

Re: [PR] feat: GHA to bump python packages using supersetbot [superset]

2024-04-11 Thread via GitHub
mistercrunch merged PR #27863: URL: https://github.com/apache/superset/pull/27863 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr

Re: [PR] fix(time_offset): improved LIMIT-handling in advanced analytics [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on code in PR #27934: URL: https://github.com/apache/superset/pull/27934#discussion_r1561315761 ## superset/common/query_context_processor.py: ## @@ -455,6 +455,13 @@ def processing_time_offsets( # pylint: disable=too-many-locals,too-many-stateme

[PR] feat(explore): Color scheme groups, new color schemes [superset]

2024-04-11 Thread via GitHub
kgabryje opened a new pull request, #27995: URL: https://github.com/apache/superset/pull/27995 ### SUMMARY - adds a bunch of new color schemes - groups the color schemes in categories: Custom (for color schemes added by users in config), Featured and Other - Adds option group handli

[PR] deps(superset-frontend): remove unused `@superset-ui/plugin-chart-period-over-period-kpi` package [superset]

2024-04-11 Thread via GitHub
corocoto opened a new pull request, #27994: URL: https://github.com/apache/superset/pull/27994 ### SUMMARY * `DEPS`: Remove `@superset-ui/plugin-chart-period-over-period-kpi` package from the dependency list (Related to #27993) ### ADDITIONAL INFORMATION Fixes #27993 --

Re: [PR] feat: optimize docker-compose up for faster boot time [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on code in PR #27953: URL: https://github.com/apache/superset/pull/27953#discussion_r1561292994 ## Dockerfile: ## @@ -119,9 +119,15 @@ RUN apt-get update -qq \ libasound2 \ libxtst6 \ wget \ -pkg-config \ -# Install G

Re: [PR] feat: optimize docker-compose up for faster boot time [superset]

2024-04-11 Thread via GitHub
mistercrunch commented on code in PR #27953: URL: https://github.com/apache/superset/pull/27953#discussion_r1561291952 ## .github/workflows/docker.yml: ## @@ -21,7 +21,7 @@ jobs: steps: - id: set_matrix run: | - MATRIX_CONFIG=$(if [ "${{ github.even

[I] Installing `superset-frontend` as a package throws an error which links to `@superset-ui/plugin-chart-period-over-period-kpi` dependency [superset]

2024-04-11 Thread via GitHub
corocoto opened a new issue, #27993: URL: https://github.com/apache/superset/issues/27993 ### Bug description I try to install `superset-frontend` folder as a package by using the [GitPkg](https://gitpkg.vercel.app) tool. But I got an error that links to `@superset-ui/plugin-chart-pe

[PR] fix(docs): add missing code formatting, fix broken link [superset]

2024-04-11 Thread via GitHub
sfirke opened a new pull request, #27992: URL: https://github.com/apache/superset/pull/27992 ### SUMMARY Adding some missing formatting to the Configuring Superset docs. This fixes BugHerd task 130: https://www.bugherd.com/projects/334204/tasks/130: > There are some unformatted bl

[I] 422 http error when List of tables refreshing [superset]

2024-04-11 Thread via GitHub
x3gluk opened a new issue, #27991: URL: https://github.com/apache/superset/issues/27991 ### Bug description If I set "Table cache timeout" in Database connection and unset it after that I will have problem then I will try to refresh table list. And in log - http 422 error. ###

Re: [PR] feat: new CSV upload form and API [superset]

2024-04-11 Thread via GitHub
michael-s-molina commented on PR #27840: URL: https://github.com/apache/superset/pull/27840#issuecomment-2049855643 > 2 - Totally agree, I think that the correct design should be imposed at the component level, then we can remove most of the custom styles.ts. So, I don't think that doing mo

Re: [PR] feat: new CSV upload form and API [superset]

2024-04-11 Thread via GitHub
dpgaspar commented on PR #27840: URL: https://github.com/apache/superset/pull/27840#issuecomment-2049848087 > Thank you for the improvements @dpgaspar. Here are the comments for the second review: > > > 1 - Fixed the display disabled at the component level > > The cursor is cor

Re: [I] [SIP-94] Support star schemas [superset]

2024-04-11 Thread via GitHub
rusackas closed issue #12729: [SIP-94] Support star schemas URL: https://github.com/apache/superset/issues/12729 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

Re: [I] [SIP-94] Support star schemas [superset]

2024-04-11 Thread via GitHub
wernerdaehn commented on issue #12729: URL: https://github.com/apache/superset/issues/12729#issuecomment-2049669612 @zhaoyongjie Superset should not allow data modeling but be able to consume data from query optimized tables. Let's see what the current method is, if you want to visual

Re: [I] [SIP-94] Support star schemas [superset]

2024-04-11 Thread via GitHub
jomach commented on issue #12729: URL: https://github.com/apache/superset/issues/12729#issuecomment-2049660526 @zhaoyongjie could you point us to the examples that you mean? I could not find a very good workaround for it. Take power bi as an example, the support this and it is very good wh

[I] Error message when trying to use mongodb connection with superset meta database functionality [superset]

2024-04-11 Thread via GitHub
KhaledBenZina opened a new issue, #27989: URL: https://github.com/apache/superset/issues/27989 ### Bug description Hello, I am running Superset on docker compose along with Apache Drill and a mongodb instance. I am trying to query the MongoDB database using Superset meta database for

[I] Users can not tag a saved query if using bulk select [superset]

2024-04-11 Thread via GitHub
EddyBess opened a new issue, #27988: URL: https://github.com/apache/superset/issues/27988 ### Bug description When I create a saved query with a non admin user and try to tag it using the bulk select option I get the following error : **1 items could not be tagged because you

Re: [I] Error on embedded dashboard after upgrading to 3.1.1: "Guest user cannot modify chart payload" [superset]

2024-04-11 Thread via GitHub
Attachai77 commented on issue #27177: URL: https://github.com/apache/superset/issues/27177#issuecomment-2049582971 I will try it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comm

Re: [I] [SIP-94] Support star schemas [superset]

2024-04-11 Thread via GitHub
zhaoyongjie commented on issue #12729: URL: https://github.com/apache/superset/issues/12729#issuecomment-2049576510 @leonliuaptean To be honest, there are lots of workarounds for the Star/Snowflake/and so on data modeling. Superset is a viz tool, and its original design didn't focus too muc

Re: [I] Error on embedded dashboard after upgrading to 3.1.1: "Guest user cannot modify chart payload" [superset]

2024-04-11 Thread via GitHub
enzo-dechaene commented on issue #27177: URL: https://github.com/apache/superset/issues/27177#issuecomment-2049573478 It fix the problem for me, can you check using Gamma role instead ? it will help to know if it's a role problem -- This is an automated message from the Apache Git Service

Re: [I] EMAIL_HEADER_MUTATOR not applying changes to MimeMultipart `msg` [superset]

2024-04-11 Thread via GitHub
smisch commented on issue #23247: URL: https://github.com/apache/superset/issues/23247#issuecomment-2049571431 Thanks for the code. I had to make some slight adjustments in 2024 to make it work: ``` import re from email.mime.multipart import MIMEMultipart from email.mime.text

Re: [I] [SIP-94] Support star schemas [superset]

2024-04-11 Thread via GitHub
leonliuaptean commented on issue #12729: URL: https://github.com/apache/superset/issues/12729#issuecomment-2049555969 we decided not to use Superset because it doesn't support Star Schema -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] fix: handle tstzrange PostgreSQL column type [superset]

2024-04-11 Thread via GitHub
dmelo closed pull request #20456: fix: handle tstzrange PostgreSQL column type URL: https://github.com/apache/superset/pull/20456 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] fix: handle tstzrange PostgreSQL column type [superset]

2024-04-11 Thread via GitHub
dmelo commented on PR #20456: URL: https://github.com/apache/superset/pull/20456#issuecomment-2049539990 Closing this PR, as it seems to have been superseded by https://github.com/apache/superset/pull/23865 . More details on the issue https://github.com/apache/superset/issues/19995 -- Th

Re: [I] Lack of support for column type tstzrange in PostgreSQL [superset]

2024-04-11 Thread via GitHub
dmelo closed issue #19995: Lack of support for column type tstzrange in PostgreSQL URL: https://github.com/apache/superset/issues/19995 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific c

Re: [I] Lack of support for column type tstzrange in PostgreSQL [superset]

2024-04-11 Thread via GitHub
dmelo commented on issue #19995: URL: https://github.com/apache/superset/issues/19995#issuecomment-2049528675 Using development mode on the master branch, with PosgreSQL 16, I can see that Superset is supporting tstzrange. I guess somebody else implemented this :smile: ![2024-04-11

[I] Any user can change the action logs using javascript [superset]

2024-04-11 Thread via GitHub
abhimittal1992 opened a new issue, #27987: URL: https://github.com/apache/superset/issues/27987 ### Bug description As a result, an attacker can insert malicious data and false entries into the logs and ultimately corrupt the file. The corrupted files can be used to cover the tracks

[I] Showing list of all the users name to other users [superset]

2024-04-11 Thread via GitHub
vasu-ram opened a new issue, #27986: URL: https://github.com/apache/superset/issues/27986 ### Bug description If any users login to the superset they can able to see the list of all the other users who didn't created or modified the dashboard In superset 3.0.1, we have 'Create

Re: [I] superset_init container fails to create example data sets [superset]

2024-04-11 Thread via GitHub
fleimgruber closed issue #27288: superset_init container fails to create example data sets URL: https://github.com/apache/superset/issues/27288 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the sp

Re: [I] superset_init container fails to create example data sets [superset]

2024-04-11 Thread via GitHub
fleimgruber commented on issue #27288: URL: https://github.com/apache/superset/issues/27288#issuecomment-2049112620 Updated podman along the way and re-tried just now. Example data sets were created. -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [I] Installing latest superset via helm, cant connect to azure redis service via ssl [superset]

2024-04-11 Thread via GitHub
shakeelansari63 commented on issue #25375: URL: https://github.com/apache/superset/issues/25375#issuecomment-2049072381 > I'm having an issue similar to @manneym. I'm upgrading to 3.1.1, from 2.1.0. Deploy through Helm chart version 0.12.7. > > Connection to Azure Cache for Redis now