Merge pull request #566 from engaging-computing/dev

Version 3.1.2

Author: kdvalin

README.md

@@ -9,10 +9,8 @@ The Engaging Computing Group develops new technologies to enable learners—yout
 ## Status
 [![CircleCI](https://circleci.com/gh/engaging-computing/MYR.svg?style=shield)](https://circleci.com/gh/engaging-computing/MYR)
 
-## Change Log - 3.1.0 -> 3.1.1
-- Patched font-size bug, previously changing font-size would clear code in the editor   
-- Dependency updates
-- nvmrc file updated (10.13 -> 12.18)
+## Change Log - 3.1.1 -> 3.1.2
+- Fixed a bug where an old scene without a name can crash the client view
 
 
 ## Acknowledgments

package.json

@@ -1,6 +1,6 @@
 {
     "name": "myr",
-    "version": "3.1.1",
+    "version": "3.1.2",
     "private": false,
     "engines": {
         "node": "^12.18.2"

src/components/structural/header/ProjectView.js

@@ -373,13 +373,15 @@ class Project extends React.Component {
  * @returns 1 if project a should come before project b, -1 if b should come before a 
  */
     projectSort = (a,b)=>{
-        if(a.name.toUpperCase() < b.name.toUpperCase())
-        {
-            return -1;
-        }
-        else if(a.name.toUpperCase() > b.name.toUpperCase())
-        {
-            return 1;
+        if(a.name && b.name) {
+            if(a.name.toUpperCase() < b.name.toUpperCase())
+            {
+                return -1;
+            }
+            else if(a.name.toUpperCase() > b.name.toUpperCase())
+            {
+                return 1;
+            }
         }
         else if(a.updateTime > b.updateTime)
         {