Re: [PR] chore(key-value): convert command to dao [superset]

2024-06-24 Thread via GitHub
codecov[bot] commented on PR #29344: URL: https://github.com/apache/superset/pull/29344#issuecomment-2186690367 ## [Codecov](https://app.codecov.io/gh/apache/superset/pull/29344?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&

[PR] fix: Excess padding in horizontal Bar charts [superset]

2024-06-24 Thread via GitHub
michael-s-molina opened a new pull request, #29345: URL: https://github.com/apache/superset/pull/29345 ### SUMMARY This PR reverts https://github.com/apache/superset/pull/27883 which incorrectly calculated the chart's padding and introduced https://github.com/apache/superset/issues/29342

Re: [I] Superset 4.0.1 - Exported CSV Contains Garbled Chinese Characters and Numbers [superset]

2024-06-24 Thread via GitHub
bionexit commented on issue #29224: URL: https://github.com/apache/superset/issues/29224#issuecomment-2186675621 +1 -- 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 unsub

Re: [PR] fix: actually write changes on "superset import-datasources" [superset]

2024-06-24 Thread via GitHub
regisb commented on PR #14817: URL: https://github.com/apache/superset/pull/14817#issuecomment-2186658272 Rebased on top of master. I'm still hoping this will be merged in 2024 :stuck_out_tongue: -- This is an automated message from the Apache Git Service. To respond to the message, plea

[PR] chore(key-value): convert command to dao [superset]

2024-06-24 Thread via GitHub
villebro opened a new pull request, #29344: URL: https://github.com/apache/superset/pull/29344 ### SUMMARY ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF ### TESTING INSTRUCTIONS ### ADDITIONAL INFORMATION - [ ] Has associated issue: - [ ]

Re: [I] [SIP-137] Can we limit a user to only read certain tables in the database connection? [superset]

2024-06-24 Thread via GitHub
andy-clapson commented on issue #29171: URL: https://github.com/apache/superset/issues/29171#issuecomment-2186549508 I think my point still applies then - if you are letting people query production databases, they should be trusted with that data (to the extent of what that particular db us

Re: [I] Excess Padding in Horizontal Bar Charts When Legend is Added [superset]

2024-06-24 Thread via GitHub
dosubot[bot] commented on issue #29342: URL: https://github.com/apache/superset/issues/29342#issuecomment-2186409477 Hi @seandiacono 👋, I'm here to assist you with any issues you encounter. I can help you solve bugs, answer questions, and even guide you on becoming a contributor.

[I] Excess Padding in Horizontal Bar Charts When Legend is Added [superset]

2024-06-24 Thread via GitHub
seandiacono opened a new issue, #29342: URL: https://github.com/apache/superset/issues/29342 ### Bug description Horizontal bar charts in Apache Superset are created with extra padding when a legend is added, affecting the chart's layout and appearance. This issue was introduced in [

Re: [I] Dashboard links are leading to Internal Error when user is not logged in [superset]

2024-06-24 Thread via GitHub
meska commented on issue #28029: URL: https://github.com/apache/superset/issues/28029#issuecomment-2186224978 I was able to pass this error with ` AUTH_ROLE_PUBLIC = "Public" ` in superset_config.py -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [I] snowflake wrong quotes on schema name with dot [superset]

2024-06-24 Thread via GitHub
dosubot[bot] commented on issue #29341: URL: https://github.com/apache/superset/issues/29341#issuecomment-2186223968 Hey @myskaludek! I'm here to help you out with any bugs, questions, or contributions you have for the repository. Let's work together to get this resolved! To

[I] snowflake wrong quotes on schema name with dot [superset]

2024-06-24 Thread via GitHub
myskaludek opened a new issue, #29341: URL: https://github.com/apache/superset/issues/29341 ### Bug description When schema name in snowflake has "." then sql lab cannot get list of tables. Schema in snowflake have for me name "out-c.superset" and in logs for listing table in s

Re: [PR] feat(reports): adding flag for removing the index on the report CSV [superset]

2024-06-24 Thread via GitHub
SkinnyPigeon commented on PR #29335: URL: https://github.com/apache/superset/pull/29335#issuecomment-2186178821 I did wonder about making the `remove_post_processing` function more general use to allow other mutations to URLs based on flags across the repo. For instance, it could be somethi

Re: [I] password changed unexpected while using api PUT /api/v1/security/users/{pk} [superset]

2024-06-24 Thread via GitHub
dosubot[bot] commented on issue #29339: URL: https://github.com/apache/superset/issues/29339#issuecomment-2185924330 Hey @q446570448! 👋 I'm here to assist you with any bugs, questions, or contributions while you wait for a human maintainer. I can help you solve issues, provide guidance,

[I] password changed unexpected while using api PUT /api/v1/security/users/{pk} [superset]

2024-06-24 Thread via GitHub
q446570448 opened a new issue, #29339: URL: https://github.com/apache/superset/issues/29339 ### Bug description my password changed unexpected while i using this api and without param : PUT /api/v1/security/users/{pk} ### How to reproduce the bug 1. login to /swagger/

[PR] fix(tags): api change tag_get_objects method to be aligned with api documentation [superset]

2024-06-24 Thread via GitHub
Bierbarbar opened a new pull request, #29338: URL: https://github.com/apache/superset/pull/29338 ### SUMMARY I just came across an issue where the implemented logic was not aligned with the API documentation on OpenAPI. As a result, the method was not usable. I modified the method to ali

Re: [I] 4.0 - Imported charts shows error on embedding [superset]

2024-06-24 Thread via GitHub
kidusmakonnen commented on issue #28446: URL: https://github.com/apache/superset/issues/28446#issuecomment-2185834119 We have also faced this issue after upgrading to 4.0.0 ![image](https://github.com/apache/superset/assets/8809047/e8f6c317-e697-4f1e-b6a6-a6e43731af9d) -- This

Re: [PR] fix: Chart cache-warmup task fails on Superset 4.0 [superset]

2024-06-24 Thread via GitHub
gempir commented on PR #28706: URL: https://github.com/apache/superset/pull/28706#issuecomment-2185756613 Is there anything I can do to help get this merged? I'm also interested in working cache warmups -- This is an automated message from the Apache Git Service. To respond to the