From 703869603eaca91e60d2ca9cad70372216caccd0 Mon Sep 17 00:00:00 2001
From: kenia <keniavalladarez@gmail.com>
Date: Wed, 14 Apr 2021 18:52:41 -0400
Subject: [PATCH] review note fixes

---
 .../app/components/topology-metrics/card/index.hbs        | 2 +-
 .../app/components/topology-metrics/down-lines/index.hbs  | 2 +-
 .../consul-ui/app/components/topology-metrics/index.js    | 2 +-
 .../app/components/topology-metrics/notice/index.hbs      | 2 +-
 .../app/components/topology-metrics/popover/index.hbs     | 8 ++++----
 .../app/components/topology-metrics/popover/index.scss    | 4 ++--
 .../consul-ui/app/components/topology-metrics/skin.scss   | 2 +-
 .../app/helpers/service/intention-permissions.js          | 2 +-
 ui/packages/consul-ui/translations/en-us.yaml             | 5 ++---
 9 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/ui/packages/consul-ui/app/components/topology-metrics/card/index.hbs b/ui/packages/consul-ui/app/components/topology-metrics/card/index.hbs
index ce64ce59b7..aaae528ad9 100644
--- a/ui/packages/consul-ui/app/components/topology-metrics/card/index.hbs
+++ b/ui/packages/consul-ui/app/components/topology-metrics/card/index.hbs
@@ -1,4 +1,4 @@
-{{#if (eq @item.Name '*(All Services)')}}
+{{#if (eq @item.Name '* (All Services)')}}
 <a class="topology-metrics-card" href={{href-to 'dc.services.index'}}>
   <p class="empty">
     {{@item.Name}}
diff --git a/ui/packages/consul-ui/app/components/topology-metrics/down-lines/index.hbs b/ui/packages/consul-ui/app/components/topology-metrics/down-lines/index.hbs
index 2d246e18a9..e5ee6a80e8 100644
--- a/ui/packages/consul-ui/app/components/topology-metrics/down-lines/index.hbs
+++ b/ui/packages/consul-ui/app/components/topology-metrics/down-lines/index.hbs
@@ -92,7 +92,7 @@
     />
   {{else if (and item.Intention.Allowed (not item.TransparentProxy) (eq item.Source 'specific-intention'))}}
     <TopologyMetrics::Popover
-      @type='notDefined'
+      @type='not-defined'
       @position={{find-by 'id' (concat this.guid item.Namespace item.Name) this.iconPositions}}
       @item={{item}}
       @oncreate={{action @oncreate item @service}}
diff --git a/ui/packages/consul-ui/app/components/topology-metrics/index.js b/ui/packages/consul-ui/app/components/topology-metrics/index.js
index cffbfd63a8..cb19b9e82b 100644
--- a/ui/packages/consul-ui/app/components/topology-metrics/index.js
+++ b/ui/packages/consul-ui/app/components/topology-metrics/index.js
@@ -73,7 +73,7 @@ export default class TopologyMetrics extends Component {
     const wildcardIntention = get(this.args.topology, 'WildcardIntention');
     if (defaultAllow || wildcardIntention) {
       items.push({
-        Name: '*(All Services)',
+        Name: '* (All Services)',
         Datacenter: '',
         Namespace: '',
         Intention: {
diff --git a/ui/packages/consul-ui/app/components/topology-metrics/notice/index.hbs b/ui/packages/consul-ui/app/components/topology-metrics/notice/index.hbs
index 00d164be25..db8ac82bbd 100644
--- a/ui/packages/consul-ui/app/components/topology-metrics/notice/index.hbs
+++ b/ui/packages/consul-ui/app/components/topology-metrics/notice/index.hbs
@@ -1,7 +1,7 @@
 <Notice
   class="topology-metrics-notice"
   ...attributes
-  @type={{or @type}}
+  @type={{@type}}
 as |notice|>
     <notice.Header>
     <h3>
diff --git a/ui/packages/consul-ui/app/components/topology-metrics/popover/index.hbs b/ui/packages/consul-ui/app/components/topology-metrics/popover/index.hbs
index 9e5d374a45..04e3316e90 100644
--- a/ui/packages/consul-ui/app/components/topology-metrics/popover/index.hbs
+++ b/ui/packages/consul-ui/app/components/topology-metrics/popover/index.hbs
@@ -47,25 +47,25 @@
       </Actions.Action>
     </:actions>
   </InformedAction>
-{{else if (eq @type 'notDefined')}}
+{{else if (eq @type 'not-defined')}}
   <InformedAction
     class="warning documentation"
     {{did-insert (set this 'popover')}}
   >
     <:header>
       <h3>
-        {{t "components.consul.topology-metrics.popover.notDefined.header"}}
+        {{t "components.consul.topology-metrics.popover.not-defined.header"}}
       </h3>
     </:header>
     <:body>
       <p>
-        {{t "components.consul.topology-metrics.popover.notDefined.body"}}
+        {{t "components.consul.topology-metrics.popover.not-defined.body"}}
       </p>
     </:body>
     <:actions as |Actions|>
       <Actions.Action class="action">
         <a href="{{env 'CONSUL_DOCS_URL'}}/connect/registration/service-registration#upstreams" rel="noopener noreferrer" target="_blank">
-          {{t "components.consul.topology-metrics.popover.notDefined.action"}}
+          {{t "components.consul.topology-metrics.popover.not-defined.action"}}
         </a>
       </Actions.Action>
       <Actions.Action>
diff --git a/ui/packages/consul-ui/app/components/topology-metrics/popover/index.scss b/ui/packages/consul-ui/app/components/topology-metrics/popover/index.scss
index deff458978..39d2aa9ae3 100644
--- a/ui/packages/consul-ui/app/components/topology-metrics/popover/index.scss
+++ b/ui/packages/consul-ui/app/components/topology-metrics/popover/index.scss
@@ -27,8 +27,8 @@
     @extend %with-layers-mask, %as-pseudo;
     background-color: $gray-300;
   }
-  &.notDefined > button::before,
-  &.notDefined .tippy-arrow::after {
+  &.not-defined > button::before,
+  &.not-defined .tippy-arrow::after {
     @extend %with-alert-triangle-mask, %as-pseudo;
     color: $yellow-500;
   }
diff --git a/ui/packages/consul-ui/app/components/topology-metrics/skin.scss b/ui/packages/consul-ui/app/components/topology-metrics/skin.scss
index f529124cdc..32b82ac737 100644
--- a/ui/packages/consul-ui/app/components/topology-metrics/skin.scss
+++ b/ui/packages/consul-ui/app/components/topology-metrics/skin.scss
@@ -65,7 +65,7 @@
     stroke: $gray-300;
     stroke-width: 2;
   }
-  path[data-permission='notDefined'] {
+  path[data-permission='not-defined'] {
     stroke-dasharray: 4;
   }
   path[data-permission='deny'] {
diff --git a/ui/packages/consul-ui/app/helpers/service/intention-permissions.js b/ui/packages/consul-ui/app/helpers/service/intention-permissions.js
index 15fc5afd74..b882fa53f9 100644
--- a/ui/packages/consul-ui/app/helpers/service/intention-permissions.js
+++ b/ui/packages/consul-ui/app/helpers/service/intention-permissions.js
@@ -11,7 +11,7 @@ export default helper(function serviceIntentionPermissions([params] /*, hash*/)
     case !allowed && !hasPermissions:
       return 'deny';
     case allowed && notExplicitlyDefined:
-      return 'notDefined';
+      return 'not-defined';
     default:
       return 'allow';
   }
diff --git a/ui/packages/consul-ui/translations/en-us.yaml b/ui/packages/consul-ui/translations/en-us.yaml
index 6afb19ed4b..021539dc91 100644
--- a/ui/packages/consul-ui/translations/en-us.yaml
+++ b/ui/packages/consul-ui/translations/en-us.yaml
@@ -202,8 +202,7 @@ components:
         not-defined-intention:
           header: Connections are not explicitly defined
           body: There appears to be an Intention defining traffic, but the services are unable to communicate until that connection is explicitly defined as a downstream or Transparent Proxy mode is turned on.
-          footer:
-            name: Read the documentation
+          footer: Read the documentation
         wildcard-intention:
           header: Permissive Intention
           body: One or more of your Intentions are set to allow traffic to and/or from all other services in a namespace. This Topology view will show all of those connections if that remains unchanged. We recommend setting more specific Intentions for upstream and downstream services to make this vizualization more useful.
@@ -223,7 +222,7 @@ components:
           action:
             isExact: Allow
             notExact: Create
-        undefined:
+        not-defined:
           header: No traffic
           body: Add the current service as an explicit upstream or turn on Transparent Proxy mode to initiate traffic.
           action: Documentation