Incubator-superset: Editing chart properties causes dashboards to become un-linked to the chart

[suddjian]

When you edit chart properties via the "edit properties" modal, it breaks any dashboards the chart appears in.

Expected results

The chart would still be associated with the dashboard

Actual results

The chart is no longer associated with the dashboard

Screenshots

These two charts were edited, changing the title and the cache timeout

How to reproduce the bug

1. Add a chart to a dashboard
2. Use the "edit properties" modal to edit the chart
3. The chart is no longer associated with the dashboard

Checklist

Make sure these boxes are checked before submitting your issue - thank you!

• [x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
• [x] I have reproduced the issue with at least the latest released version of superset.
• [x] I have checked the issue tracker for the same issue and I haven't found one similar.

[issue-label-bot[bot]]

Issue-Label Bot is automatically applying the label #bug to this issue, with a confidence of 0.97. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

[stale[bot]]

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.

[junlincc]

https://github.com/apache/incubator-superset/issues/11623

@kgabryje let's address these two issues together

[mistercrunch]

Is this still an issue?

[junlincc]

yes, i tested when i filed #11623 @mistercrunch

[kgabryje]

On it

[kgabryje]

I think it's a backend bug. After making a PUT request to /api/v1/chart/${slice_id} , the modified chart is not returned in the response to Dashboard view (or rather in data-bootstrap which is server-rendered).
@kkucharc Could you take a look?