Add character limit to sketch name on the back end #568

joeyfurness · joeyfurness · commit db9a703def0a · 2020-08-03T11:40:48.000-07:00
diff --git a/server/controllers/project.controller/createProject.js b/server/controllers/project.controller/createProject.js
@@ -8,8 +8,9 @@ export default function createProject(req, res) {
 
   projectValues = Object.assign(projectValues, req.body);
 
+  // TODO: Error handling to match spec
   function sendFailure() {
-    res.json({ success: false });
+    res.status(422).json({ success: false });
   }
 
   function populateUserData(newProject) {
diff --git a/server/models/project.js b/server/models/project.js
@@ -29,7 +29,7 @@ fileSchema.set('toJSON', {
 
 const projectSchema = new Schema(
   {
-    name: { type: String, default: "Hello p5.js, it's the server" },
+    name: { type: String, default: "Hello p5.js, it's the server", maxlength: 256 },
     user: { type: Schema.Types.ObjectId, ref: 'User' },
     serveSecure: { type: Boolean, default: false },
     files: { type: [fileSchema] },

Original file line number	Diff line number	Diff line change
`@@ -8,8 +8,9 @@ export default function createProject(req, res) {`
`8`	`8`
`9`	`9`	`projectValues = Object.assign(projectValues, req.body);`
`10`	`10`
	`11`	`+ // TODO: Error handling to match spec`
`11`	`12`	`function sendFailure() {`
`12`		`- res.json({ success: false });`
	`13`	`+ res.status(422).json({ success: false });`
`13`	`14`	`}`
`14`	`15`
`15`	`16`	`function populateUserData(newProject) {`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ fileSchema.set('toJSON', {`
`29`	`29`
`30`	`30`	`const projectSchema = new Schema(`
`31`	`31`	`{`
`32`		`- name: { type: String, default: "Hello p5.js, it's the server" },`
	`32`	`+ name: { type: String, default: "Hello p5.js, it's the server", maxlength: 256 },`
`33`	`33`	`user: { type: Schema.Types.ObjectId, ref: 'User' },`
`34`	`34`	`serveSecure: { type: Boolean, default: false },`
`35`	`35`	`files: { type: [fileSchema] },`