fix: link to docs on the monitor tab (#929)

* fix: link to docs on the monitor tab This commit fixes a broken link on the empty state banner of the monitor page. Signed-off-by: Alessandro Racheli <alessandroracheli@gmail.com> * fix: snapshot tests This commit updates the related snapshot tests. Signed-off-by: Alessandro Racheli <alessandroracheli@gmail.com>
jaegertracing · Apr 13, 2022 · 5d05a79 · 5d05a79
1 parent 53da5e6
commit 5d05a79
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/jaeger-ui/src/components/Monitor/ServicesView/__snapshots__/index.test.js.snap b/packages/jaeger-ui/src/components/Monitor/ServicesView/__snapshots__/index.test.js.snap
@@ -371,7 +371,11 @@ exports[`<MonitorATMServicesView> ATM snapshot test with no metrics 1`] = `
         <Link
           replace={false}
           target="_blank"
-          to="//https://www.jaegertracing.io/docs/latest/spm/"
+          to={
+            Object {
+              "pathname": "https://www.jaegertracing.io/docs/latest/spm/",
+            }
+          }
         >
            instructions 
         </Link>

diff --git a/packages/jaeger-ui/src/components/Monitor/ServicesView/index.tsx b/packages/jaeger-ui/src/components/Monitor/ServicesView/index.tsx
@@ -243,7 +243,7 @@ export class MonitorATMServicesViewImpl extends React.PureComponent<TProps, Stat
             message={
               <>
                 No data yet! Please see these
-                <Link to={`//${this.docsLink}`} target="_blank">
+                <Link to={{ pathname: this.docsLink }} target="_blank">
                   &nbsp;instructions&nbsp;
                 </Link>
                 on how to set up your span metrics.