PreviousEnvironments and the long link (#62)

* PreviousEnvironments and the long link

Fixes #61

* correct key usage

* console.log leftovere

Author: Peter Bengtsson

app.py

@@ -326,22 +326,12 @@ def get(self):
         environments = []
         shortlinks = Shortlink.query.filter(Shortlink.link.in_(ids)).all()
         for shortlink in shortlinks:
-            qs = {
-                "repo": shortlink.repo,
-                "owner": shortlink.owner,
-                "name[]": [],
-                "url[]": [],
-            }
-            for k, v in json.loads(shortlink.revisions):
-                qs["name[]"].append(k)
-                qs["url[]"].append(v)
-            url = "/?" + urlencode(qs, True)
             environments.append(
                 {
+                    "shortlink": shortlink.link,
                     "owner": shortlink.owner,
                     "repo": shortlink.repo,
                     "revisions": json.loads(shortlink.revisions),
-                    "url": url,
                 }
             )
 

src/App.js

@@ -1,5 +1,5 @@
 import React from 'react';
-import { BrowserRouter as Router, NavLink } from 'react-router-dom';
+import { BrowserRouter as Router, Link } from 'react-router-dom';
 import 'bootstrap/dist/css/bootstrap.min.css';
 
 import Routes from './Routes';
@@ -23,7 +23,7 @@ class Header extends React.Component {
   render() {
     return (
       <h2 className="text-center">
-        <NavLink to="/">What's Deployed?</NavLink>
+        <Link to="/">What's Deployed?</Link>
       </h2>
     );
   }

src/SetupPage.js

@@ -1,5 +1,5 @@
 import React from 'react';
-import { withRouter } from 'react-router-dom';
+import { withRouter, Link } from 'react-router-dom';
 
 import shortUrls from './shortUrls';
 
@@ -144,13 +144,10 @@ class SetupForm extends React.Component {
 const SetupFormWithRouter = withRouter(SetupForm);
 
 class PreviousEnvironments extends React.Component {
-  constructor(props) {
-    super(props);
-    this.state = {
-      environments: [],
-      loading: false
-    };
-  }
+  state = {
+    environments: [],
+    loading: false
+  };
 
   componentWillUnmount() {
     this.dismounted = true;
@@ -177,13 +174,13 @@ class PreviousEnvironments extends React.Component {
 
     return (
       <div id="previous">
-        <h3>PreviousEnvironments</h3>
+        <h3>Previous Environments</h3>
         <ul>
           {environments.map(env => (
-            <li key={env.revisions[0]}>
-              <a href={env.url}>
+            <li key={env.shortlink}>
+              <Link to={`/s/${env.shortlink}`}>
                 {env.owner}/{env.repo}
-              </a>
+              </Link>
               <span className="names">
                 {env.revisions.map(r => r[0]).join(', ')}
               </span>