sync(branch 18.0): merge tag fixes for tk_construction_management + ks_dashboard_ninja
All checks were successful
addon-qualify / qualify (push) Successful in 13s
All checks were successful
addon-qualify / qualify (push) Successful in 13s
The Pillar 1 CI gate flagged drift between tag state and branch HEAD on these two addons. Today (2026-05-09) we force-tagged fixes for: - tk_construction_management/18.0.2.0.8: add name= to <app>, chatter migration to <chatter/>, chart NaN guard, scope .o_action_manager CSS rule, remove dasdsa debug logs. - ks_dashboard_ninja/18.0.1.1.7: rename webpackChunk_am5 to webpackChunk_am5_ksdn so it does not collide with synconics_bi_dashboard. Replicating the same content on the 18.0 branch HEAD so future pushes do not silently revert these fixes.
This commit is contained in:
@@ -1,7 +1,5 @@
|
||||
.o_action_manager {
|
||||
overflow: auto !important;
|
||||
}
|
||||
.construction_dashboard {
|
||||
overflow: auto !important;
|
||||
background-color: #f8f9fa !important;
|
||||
height: 100%;
|
||||
.d-flex {
|
||||
|
||||
@@ -45,8 +45,6 @@ export class ConstructionDashboard extends Component {
|
||||
// args: [false, false],
|
||||
// });
|
||||
const result = await this.orm.call("tk.construction.dashboard", "get_construction_state", [false, false]);
|
||||
console.log(result,'dasdsa');
|
||||
|
||||
this.state.stats = result;
|
||||
this.state.sites = Object.entries(result.con_sites || {}).map(([id, name]) => ({ id, name }));
|
||||
this.renderCharts(result);
|
||||
@@ -82,7 +80,6 @@ export class ConstructionDashboard extends Component {
|
||||
|
||||
this.state.stats = data;
|
||||
this.renderCharts(data);
|
||||
console.log(this.state.stats,'dasdsa');
|
||||
}
|
||||
|
||||
openAction(name, resModel, domain = []) {
|
||||
@@ -221,6 +218,16 @@ export class ConstructionDashboard extends Component {
|
||||
renderGraph(el, options) {
|
||||
if (!el) return;
|
||||
el.innerHTML = "";
|
||||
// NaN-guard: ApexCharts emits 'M NaN NaN A NaN' SVG warnings when series is
|
||||
// empty or all zeros (donut/pie divide by total=0). Show 'No data' instead.
|
||||
const series = options.series || [];
|
||||
const hasData = series.some((s) =>
|
||||
typeof s === 'number' ? s > 0 : (s && s.data && s.data.length > 0)
|
||||
);
|
||||
if (!hasData) {
|
||||
el.innerHTML = '<div class="text-muted text-center p-4">No data</div>';
|
||||
return;
|
||||
}
|
||||
const chart = new ApexCharts(el, options);
|
||||
chart.render();
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user