Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Edit the default "No Results" message #9722

Closed
larsmars opened this issue May 4, 2020 · 13 comments
Closed

Edit the default "No Results" message #9722

larsmars opened this issue May 4, 2020 · 13 comments
Labels
enhancement:request Enhancement request submitted by anyone from the community need:followup Requires followup

Comments

@larsmars
Copy link

larsmars commented May 4, 2020

Is your feature request related to a problem? Please describe.
When superset chart is used as iFrame make it posible to edit the default "No Results" message to you pref.

Default text today:
No Results
No results were returned for this query. If you expected results to be returned, ensure any filters are configured properly and the datasource contains data for the selected time range.

Describe the solution you'd like
Config parameter to edit default text and header for "No Results"

Additional context
Screenshot of as is today:
no_results

@larsmars larsmars added the enhancement:request Enhancement request submitted by anyone from the community label May 4, 2020
@stale
Copy link

stale bot commented Jul 3, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue .pinned to prevent stale bot from closing the issue.

@stale stale bot added the inactive Inactive for >= 30 days label Jul 3, 2020
@larsmars
Copy link
Author

larsmars commented Jul 3, 2020

bump

@stale stale bot removed the inactive Inactive for >= 30 days label Jul 3, 2020
@mistercrunch
Copy link
Member

There's been some design work between Airbnb and Preset to redesign error messages from the ground up. I don't think that work is available in the open just yet, but there's a plan to make error messages much more clear and actionable.

@sarahbarberuk
Copy link

I'm also interested in this

@Neel-rishabhsoft
Copy link

I am as well interested. But, meanwhile they become open is there anything I can do alter this default message temporarily?
I have been stuck on changing this for weeks, any suggestion would be really helpful.
Thanks in advance.

@Neel-rishabhsoft
Copy link

Hi @mistercrunch, thanks for the reply.
I am sorry if I sound really daft. But I am not able to detect how this superset-ui plugins are connect with core "incubator-superset" code. I have searched the libraries getting installed with Superset & superset-ui doesnt get installed with it. So I am guessing there must be a connection point somewhere in the core code.
Can you please give me some direction in which I can move to alter the superset-ui code?

@stale
Copy link

stale bot commented Nov 26, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue .pinned to prevent stale bot from closing the issue.

@stale stale bot added the inactive Inactive for >= 30 days label Nov 26, 2020
@baltom
Copy link

baltom commented Dec 17, 2020

I'm also interested in this. Would be a nice feature

@stale stale bot removed the inactive Inactive for >= 30 days label Dec 17, 2020
@lucaxsilveira
Copy link

I'm also interested in this.

@junlincc
Copy link
Member

@baltom @lucaxsilveira @Neel-rishabhsoft thanks for interested in contributing!

@villebro @ktmud could you give some pointers where/how folks from community can make this change? thanks!

@junlincc junlincc added the need:followup Requires followup label Feb 16, 2021
@ktmud
Copy link
Member

ktmud commented Feb 16, 2021

As Max noted, currently this text is hard-coded in a superset-ui component, which, like all superset-ui changes, is not every easy to update.

I'm hesitant to add another config value just for this message, but we can maybe either

  1. Move SuperChart or the no-results file out of superset-ui to make it easier to override
  2. Handle no results like regular frontend errors, so we can reuse the error message registry introduced by @etr2460 , or
  3. Wait for monorepo migration ([SIP-58] Proposal to move superset-ui to this repo #13013)

@maltoze
Copy link
Contributor

maltoze commented Jun 11, 2021

If this pr(apache-superset/superset-ui#1144) is merged, we can alter this message temporarily by updating the translation file.

@apache apache locked and limited conversation to collaborators Feb 2, 2022
@geido geido converted this issue into discussion #18387 Feb 2, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
enhancement:request Enhancement request submitted by anyone from the community need:followup Requires followup
Projects
None yet
Development

No branches or pull requests

9 participants