Code Review by David

imports/ui/About.jsx

@@ -13,6 +13,7 @@ class About extends Component {
                    About
                   </span>
                 </h2>
+                {/* should escape quote here */}
                 <h5>Dixit is a card game using a deck of cards illustrated with dreamlike images, 
                      players select cards that match a title suggestsed by the "storyteller", and 
                      attempt to vote which card the "storyteller" selected...</h5> 

imports/ui/App.jsx

@@ -35,9 +35,11 @@ class App extends Component {
         <div className = "container">
               <Switch>
                 <Route exact path="/" component={HomePage} />
+                {/* How to get to this page */}
                 <Route exact path="/AddCard" component={AddCard} />
                  <Route exact path="/About" component={About} />
                 <Fragment>
+                  {/* You can Do something : <Route exact path="/gameroom" component= { this.props.myData.length === 0 ? GameRoom : this.props.myData[0].ingame ? MyGame : GameRoom} /> */}
                   {this.props.myData.length === 0 ? <Route exact path="/gameroom" component= {GameRoom} /> : <div>
                     {this.props.myData[0].ingame ? <Route exact path="/gameroom" component={MyGame} /> :
                       <Route exact path="/gameroom" component= {GameRoom} />}</div>

imports/ui/HomePage.jsx

@@ -4,7 +4,7 @@ class HomePage extends Component {
   render() {
     return (
         <div className="jumbotron"> 
-
+          {/* Here you better have something in the landing page */}
         </div>
     );
   }

imports/ui/MyGame.jsx

@@ -475,6 +475,7 @@ class MyGame extends Component {
               </div>
                 : 
                 <div>
+                  {/* TargetCard might be null */}
                   {this.state.stage === 2 && this.state.isHost? 
                     <div className="row">
                       <h4>
@@ -660,7 +661,7 @@ MyGame.propTypes = {
   cards: PropTypes.arrayOf(PropTypes.object).isRequired,
   ready: PropTypes.bool.isRequired
 };
-
+/* Is it better to pass myGame from App into MyGame? */
 export default withTracker(() => {
   const handle = Meteor.subscribe("myGame");
   const handle2 = Meteor.subscribe("gameData");