Fix translation loading and manifest fetching for integration page (#…

…21501)
home-assistant · Jul 30, 2024 · a5786b4 · a5786b4
1 parent 4ade395
commit a5786b4
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 4 deletions.
diff --git a/src/layouts/hass-tabs-subpage-data-table.ts b/src/layouts/hass-tabs-subpage-data-table.ts
@@ -266,7 +266,8 @@ export class HaTabsSubpageDataTable extends LitElement {
           <ha-assist-chip
             .label=${localize("ui.components.subpage-data-table.sort_by", {
               sortColumn: this._sortColumn
-                ? ` ${this.columns[this._sortColumn].title || this.columns[this._sortColumn].label}`
+                ? ` ${this.columns[this._sortColumn]?.title || this.columns[this._sortColumn]?.label}` ||
+                  ""
                 : "",
             })}
             id="sort-by-anchor"

diff --git a/src/panels/config/integrations/ha-config-integration-page.ts b/src/panels/config/integrations/ha-config-integration-page.ts
@@ -538,7 +538,9 @@ class HaConfigIntegrationPage extends SubscribeMixin(LitElement) {
               </h1>
               ${normalEntries.length === 0
                 ? html`<div class="card-content no-entries">
-                    ${this.hass.config.components.find(
+                    ${this._manifest &&
+                    !this._manifest.config_flow &&
+                    this.hass.config.components.find(
                       (comp) => comp.split(".")[0] === this.domain
                     )
                       ? this.hass.localize(

diff --git a/src/panels/config/integrations/ha-config-integrations-dashboard.ts b/src/panels/config/integrations/ha-config-integrations-dashboard.ts
@@ -172,6 +172,7 @@ class HaConfigIntegrationsDashboard extends SubscribeMixin(LitElement) {
         if (
           !entryDomains.has(componentDomain) &&
           manifests[componentDomain] &&
+          !manifests[componentDomain].config_flow &&
           (!manifests[componentDomain].integration_type ||
             ["device", "hub", "service", "integration"].includes(
               manifests[componentDomain].integration_type!
@@ -314,6 +315,11 @@ class HaConfigIntegrationsDashboard extends SubscribeMixin(LitElement) {
         this.configEntriesInProgress.map((flow) => flow.handler)
       );
     }
+    if (changed.has("configEntries") && this.configEntries) {
+      this._fetchIntegrationManifests(
+        this.configEntries.map((entry) => entry.domain)
+      );
+    }
   }
 
   protected render() {

diff --git a/src/panels/config/integrations/ha-config-integrations.ts b/src/panels/config/integrations/ha-config-integrations.ts
@@ -151,7 +151,10 @@ class HaConfigIntegrations extends SubscribeMixin(HassRouterPage) {
     if (this.hasUpdated) {
       return;
     }
-    this._loadTranslationsPromise = this.hass.loadBackendTranslation("title");
+    this._loadTranslationsPromise = this.hass.loadBackendTranslation(
+      "title",
+      this.hass.config.components.map((comp) => comp.split(".")[0])
+    );
   }
 
   protected updatePageEl(pageEl) {

diff --git a/src/panels/config/integrations/ha-integration-card.ts b/src/panels/config/integrations/ha-integration-card.ts
@@ -182,7 +182,7 @@ export class HaIntegrationCard extends LitElement {
                 >
               </div>`
             : nothing}
-          ${!this.manifest?.config_flow
+          ${this.manifest && !this.manifest?.config_flow
             ? html`<div class="icon yaml">
                 <ha-svg-icon .path=${mdiCodeBraces}></ha-svg-icon>
                 <simple-tooltip