fix: 🐛 Fix adding host set error in targets (#2267)

ui/admin/app/routes/scopes/scope/targets/target/add-brokered-credential-sources.js

@@ -37,6 +37,12 @@ export default class ScopesScopeTargetsTargetAddBrokeredCredentialSourcesRoute e
       scope_id,
       query: { filters: { scope_id: [{ equals: scope_id }] } },
     });
+
+    // TODO: For some reason, not returning promises fixes
+    //  an ember bug similar to this reported issue:
+    //  https://github.com/emberjs/data/issues/8299.
+    //  This is a temporary fix until we can find a better solution or
+    //  we upgrade ember data to try to fix the issue.
     await Promise.all(
       credentialStores.map(({ id: credential_store_id, isStatic }) => {
         if (isStatic) {

ui/admin/app/routes/scopes/scope/targets/target/add-host-sources.js

@@ -6,7 +6,6 @@
 import Route from '@ember/routing/route';
 import { inject as service } from '@ember/service';
 import { action } from '@ember/object';
-import { all } from 'rsvp';
 import { loading } from 'ember-loading';
 import { notifySuccess, notifyError } from 'core/decorators/notify';
 
@@ -37,10 +36,16 @@ export default class ScopesScopeTargetsTargetAddHostSourcesRoute extends Route {
       scope_id,
       query: { filters: { scope_id: [{ equals: scope_id }] } },
     });
-    await all(
-      hostCatalogs.map(({ id: host_catalog_id }) =>
-        this.store.query('host-set', { host_catalog_id }),
-      ),
+
+    // TODO: For some reason, not returning promises fixes
+    //  an ember bug similar to this reported issue:
+    //  https://github.com/emberjs/data/issues/8299.
+    //  This is a temporary fix until we can find a better solution or
+    //  we upgrade ember data to try to fix the issue.
+    await Promise.all(
+      hostCatalogs.map(({ id: host_catalog_id }) => {
+        this.store.query('host-set', { host_catalog_id });
+      }),
     );
     const hostSets = this.store.peekAll('host-set');
     return {

ui/admin/app/routes/scopes/scope/targets/target/add-injected-application-credential-sources.js

@@ -37,6 +37,12 @@ export default class ScopesScopeTargetsTargetAddInjectedApplicationCredentialSou
       scope_id,
       query: { filters: { scope_id: [{ equals: scope_id }] } },
     });
+
+    // TODO: For some reason, not returning promises fixes
+    //  an ember bug similar to this reported issue:
+    //  https://github.com/emberjs/data/issues/8299.
+    //  This is a temporary fix until we can find a better solution or
+    //  we upgrade ember data to try to fix the issue.
     await Promise.all(
       credentialStores.map(({ id: credential_store_id, isStatic }) => {
         if (isStatic) {