FelipeGLopez commented on code in PR #36708:
URL: https://github.com/apache/superset/pull/36708#discussion_r2657841908
##########
superset-frontend/src/explore/exploreUtils/index.js:
##########
@@ -351,3 +352,18 @@ export const getSimpleSQLExpression = (subject, operator,
comparator) => {
export function formatSelectOptions(options) {
return options.map(opt => [opt, opt.toString()]);
}
+
+/**
+ * Fetches the latest chart configuration from the database
+ * @param {number} chartId - The chart ID to fetch
+ * @returns {Promise<Object>} - Promise resolving to the chart's form_data
+ */
+export async function fetchChartFormData(chartId) {
Review Comment:
I checked and it doesn't exist. Functions like `fetchCssTemplates` or
`formatCurrentQuery` they apply the code directly in the same component.
Probably I should do the same here and replace this code:
```
...
try {
formDataToUse = await fetchChartFormData(chart.id);
} catch (error) {
....
```
by the content of that function (fetchChartFormData) and delete it from
exploreUtils? (this would probably lead to rewrite the tests as well).
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
