venkatamandavilli-code commented on issue #40192: URL: https://github.com/apache/superset/issues/40192#issuecomment-4471291500
Thanks for reporting and analyzing this issue. The root cause explanation makes sense, especially around `dashboardInfo` becoming partially synchronized during layout edits. The proposed fix to only include `certified_by` and `certification_details` when explicitly defined looks safer than sending empty strings in the PUT payload. That approach should help avoid unintentionally overwriting existing certification metadata during dashboard save operations. It may also be useful to add a regression test covering: * dashboard certification creation * layout modification * dashboard save * verification that certification fields remain preserved after reload This looks like an important fix since certification metadata is often used for governance and trusted dashboard identification. -- 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]
