From 7696be5d95c576fd2e68f0c0d2817b916c4e2669 Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Thu, 3 Nov 2022 22:00:45 -0500
Subject: [PATCH 01/13] first commit

---
 .../editor/metadataEditor/metadataEditor.jsx  | 35 ++++++++++++++++---
 1 file changed, 30 insertions(+), 5 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index 25b4daac8..91da62aa0 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -14,6 +14,17 @@ const SYSTEMS = ['5e', '4e', '3.5e', 'Pathfinder'];
 
 const homebreweryThumbnail = require('../../thumbnail.png');
 
+const validators = {
+	thumbnail : [
+		(value)=>{
+			return value?.length > 5 ? 'Max URL length of 5 characters' : null;
+		},
+		(value)=>{
+			return (value ?? '')[0] !== 'W' ? 'URL must start with W' : null;
+		}
+	]
+};
+
 const MetadataEditor = createClass({
 	displayName     : 'MetadataEditor',
 	getDefaultProps : function() {
@@ -35,7 +46,8 @@ const MetadataEditor = createClass({
 
 	getInitialState : function(){
 		return {
-			showThumbnail : true
+			showThumbnail   : true,
+			validationError : null
 		};
 	},
 
@@ -51,11 +63,23 @@ const MetadataEditor = createClass({
 	},
 
 	handleFieldChange : function(name, e){
-		this.props.onChange({
-			...this.props.metadata,
-			[name] : e.target.value
-		});
+		const inputRules = validators[name] ?? [];
+		const validationErr = inputRules.map((rule)=>rule(e.target.value)).filter(Boolean);
+		this.setState((prevState)=>({
+		    validationError : {
+				...(prevState.validationError ?? {}),
+				[name] : validationErr.length > 0 ? validationErr : undefined
+			}
+		}));
+		if(Object.values(this.state.validationError ?? {}).filter(Boolean).length === 0){
+			this.props.onChange({
+				...this.props.metadata,
+				[name] : e.target.value
+			});
+		}
+
 	},
+
 	handleSystem : function(system, e){
 		if(e.target.checked){
 			this.props.metadata.systems.push(system);
@@ -64,6 +88,7 @@ const MetadataEditor = createClass({
 		}
 		this.props.onChange(this.props.metadata);
 	},
+
 	handleRenderer : function(renderer, e){
 		if(e.target.checked){
 			this.props.metadata.renderer = renderer;

From 589ec0251a1bd42af27276bf6dd4039dce434cba Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Fri, 4 Nov 2022 15:32:20 -0500
Subject: [PATCH 02/13] move validations into it's own file.

---
 .../editor/metadataEditor/metadataEditor.jsx  | 27 +++++++------------
 .../editor/metadataEditor/validations.js      | 23 ++++++++++++++++
 2 files changed, 33 insertions(+), 17 deletions(-)
 create mode 100644 client/homebrew/editor/metadataEditor/validations.js

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index 91da62aa0..0ae308fc8 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -9,21 +9,13 @@ const Nav = require('naturalcrit/nav/nav.jsx');
 const StringArrayEditor = require('../stringArrayEditor/stringArrayEditor.jsx');
 
 const Themes = require('themes/themes.json');
+const validations = require('./validations.js')
 
 const SYSTEMS = ['5e', '4e', '3.5e', 'Pathfinder'];
 
 const homebreweryThumbnail = require('../../thumbnail.png');
 
-const validators = {
-	thumbnail : [
-		(value)=>{
-			return value?.length > 5 ? 'Max URL length of 5 characters' : null;
-		},
-		(value)=>{
-			return (value ?? '')[0] !== 'W' ? 'URL must start with W' : null;
-		}
-	]
-};
+
 
 const MetadataEditor = createClass({
 	displayName     : 'MetadataEditor',
@@ -33,6 +25,7 @@ const MetadataEditor = createClass({
 				editId      : null,
 				title       : '',
 				description : '',
+				thumbnail   : '',
 				tags        : [],
 				published   : false,
 				authors     : [],
@@ -46,8 +39,8 @@ const MetadataEditor = createClass({
 
 	getInitialState : function(){
 		return {
-			showThumbnail   : true,
-			validationError : null
+			showThumbnail : true,
+			errs          : null
 		};
 	},
 
@@ -63,20 +56,20 @@ const MetadataEditor = createClass({
 	},
 
 	handleFieldChange : function(name, e){
-		const inputRules = validators[name] ?? [];
+		const inputRules = validations[name] ?? [];
 		const validationErr = inputRules.map((rule)=>rule(e.target.value)).filter(Boolean);
 		this.setState((prevState)=>({
-		    validationError : {
-				...(prevState.validationError ?? {}),
+		    errs : {
+				...(prevState.errs ?? {}),
 				[name] : validationErr.length > 0 ? validationErr : undefined
 			}
 		}));
-		if(Object.values(this.state.validationError ?? {}).filter(Boolean).length === 0){
+		if(Object.values(this.state.errs ?? {}).filter(Boolean).length === 0){
 			this.props.onChange({
 				...this.props.metadata,
 				[name] : e.target.value
 			});
-		}
+		};
 
 	},
 
diff --git a/client/homebrew/editor/metadataEditor/validations.js b/client/homebrew/editor/metadataEditor/validations.js
new file mode 100644
index 000000000..48d037b54
--- /dev/null
+++ b/client/homebrew/editor/metadataEditor/validations.js
@@ -0,0 +1,23 @@
+module.exports = {
+	title : [
+		(value)=>{
+			return value?.length > 10 ? 'Max URL length of 10 characters' : null;
+		}
+	],
+	description : [
+		(value)=>{
+			return value?.length > 10 ? 'Max URL length of 10 characters' : null;
+		}
+	],
+	thumbnail : [
+		(value)=>{
+			return value?.length > 5 ? 'Max URL length of 5 characters' : null;
+		},
+		(value)=>{
+			return (value ?? '')[0] !== 'W' ? 'URL must start with W' : null;
+		}
+	]
+};
+
+
+

From 8c52a253dce34427152c3b6757126574093b38e9 Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Fri, 4 Nov 2022 20:22:59 -0500
Subject: [PATCH 03/13] log errs correctly, setCustomValidity and
 reportValidity

---
 .../editor/metadataEditor/metadataEditor.jsx   | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index 0ae308fc8..76454e88b 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -65,10 +65,17 @@ const MetadataEditor = createClass({
 			}
 		}));
 		if(Object.values(this.state.errs ?? {}).filter(Boolean).length === 0){
+			e.target.setCustomValidity('');
 			this.props.onChange({
 				...this.props.metadata,
 				[name] : e.target.value
 			});
+		} else {
+			const errMessage = this.state.errs[name].map((err)=>{
+				return `- ${err}`;
+			}).join('\n');
+			e.target.setCustomValidity(errMessage);
+			e.target.reportValidity();
 		};
 
 	},
@@ -241,25 +248,30 @@ const MetadataEditor = createClass({
 		</div>;
 	},
 
+	// renderErrors : function(){
+	// 	if(Object.values(this.state.errs ?? {}).filter(Boolean).length > 0) return;
+	// 	return <span className='validation-error'></span>
+	// },
+
 	render : function(){
 		return <div className='metadataEditor'>
 			<div className='field title'>
 				<label>title</label>
 				<input type='text' className='value'
-					value={this.props.metadata.title}
+					defaultValue={this.props.metadata.title}
 					onChange={(e)=>this.handleFieldChange('title', e)} />
 			</div>
 			<div className='field-group'>
 				<div className='field-column'>
 					<div className='field description'>
 						<label>description</label>
-						<textarea value={this.props.metadata.description} className='value'
+						<textarea defaultValue={this.props.metadata.description} className='value'
 							onChange={(e)=>this.handleFieldChange('description', e)} />
 					</div>
 					<div className='field thumbnail'>
 						<label>thumbnail</label>
 						<input type='text'
-							value={this.props.metadata.thumbnail}
+							defaultValue={this.props.metadata.thumbnail}
 							placeholder='my.thumbnail.url'
 							className='value'
 							onChange={(e)=>this.handleFieldChange('thumbnail', e)} />

From edce191c2961992db31ad31e695f83e261bd84ef Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Sat, 5 Nov 2022 11:32:35 -0500
Subject: [PATCH 04/13] remove a silly rule

---
 client/homebrew/editor/metadataEditor/validations.js | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/validations.js b/client/homebrew/editor/metadataEditor/validations.js
index 48d037b54..0023c6008 100644
--- a/client/homebrew/editor/metadataEditor/validations.js
+++ b/client/homebrew/editor/metadataEditor/validations.js
@@ -12,9 +12,6 @@ module.exports = {
 	thumbnail : [
 		(value)=>{
 			return value?.length > 5 ? 'Max URL length of 5 characters' : null;
-		},
-		(value)=>{
-			return (value ?? '')[0] !== 'W' ? 'URL must start with W' : null;
 		}
 	]
 };

From 78da48b08dc2200b0d4e0b42a5b65917bd135123 Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Sat, 5 Nov 2022 11:42:21 -0500
Subject: [PATCH 05/13] fix issue with async setState using `Event.persist()`

---
 .../editor/metadataEditor/metadataEditor.jsx  | 31 ++++++++++---------
 1 file changed, 17 insertions(+), 14 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index 76454e88b..e095d1c6c 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -56,6 +56,7 @@ const MetadataEditor = createClass({
 	},
 
 	handleFieldChange : function(name, e){
+		e.persist();
 		const inputRules = validations[name] ?? [];
 		const validationErr = inputRules.map((rule)=>rule(e.target.value)).filter(Boolean);
 		this.setState((prevState)=>({
@@ -63,20 +64,22 @@ const MetadataEditor = createClass({
 				...(prevState.errs ?? {}),
 				[name] : validationErr.length > 0 ? validationErr : undefined
 			}
-		}));
-		if(Object.values(this.state.errs ?? {}).filter(Boolean).length === 0){
-			e.target.setCustomValidity('');
-			this.props.onChange({
-				...this.props.metadata,
-				[name] : e.target.value
-			});
-		} else {
-			const errMessage = this.state.errs[name].map((err)=>{
-				return `- ${err}`;
-			}).join('\n');
-			e.target.setCustomValidity(errMessage);
-			e.target.reportValidity();
-		};
+		}), ()=>{
+			if(Object.values(this.state.errs ?? {}).filter(Boolean).length === 0){
+				e.target.setCustomValidity('');
+				this.props.onChange({
+					...this.props.metadata,
+					[name] : e.target.value
+				});
+			} else {
+				const errMessage = this.state.errs[name].map((err)=>{
+					return `- ${err}`;
+				}).join('\n');
+				e.target.setCustomValidity(errMessage);
+				e.target.reportValidity();
+			};
+		});
+
 
 	},
 

From aec1147aa53af84ebe128b0e0fe56117b7d4e011 Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Sat, 5 Nov 2022 11:56:48 -0500
Subject: [PATCH 06/13] set `:invalid` input to light red background

---
 client/homebrew/editor/metadataEditor/metadataEditor.less | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.less b/client/homebrew/editor/metadataEditor/metadataEditor.less
index 9b2761306..a852ad41b 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.less
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.less
@@ -42,6 +42,9 @@
 		&>.value{
 			flex           : 1 1 auto;
 			width          : 50px;
+			&:invalid {
+				background : #ffb9b9;
+			}
 		}
 		&.thumbnail{
 			height      : 1.4em;

From a13fd48cdae9d3ee5eea1c90801c160b3d983c47 Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Sat, 5 Nov 2022 12:19:01 -0500
Subject: [PATCH 07/13] prevent crash on input to second field. small css
 change

---
 client/homebrew/editor/metadataEditor/metadataEditor.jsx  | 2 +-
 client/homebrew/editor/metadataEditor/metadataEditor.less | 3 +++
 client/homebrew/editor/metadataEditor/validations.js      | 4 ++--
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index e095d1c6c..47110c097 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -65,7 +65,7 @@ const MetadataEditor = createClass({
 				[name] : validationErr.length > 0 ? validationErr : undefined
 			}
 		}), ()=>{
-			if(Object.values(this.state.errs ?? {}).filter(Boolean).length === 0){
+			if(Object.values(this.state.errs ?? {}).filter(Boolean).length === 0 || !this.state.errs[name]){
 				e.target.setCustomValidity('');
 				this.props.onChange({
 					...this.props.metadata,
diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.less b/client/homebrew/editor/metadataEditor/metadataEditor.less
index a852ad41b..f0974fb6c 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.less
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.less
@@ -46,6 +46,9 @@
 				background : #ffb9b9;
 			}
 		}
+		input[type='text'], textarea {
+			border : 1px solid gray;
+		}
 		&.thumbnail{
 			height      : 1.4em;
 			label{
diff --git a/client/homebrew/editor/metadataEditor/validations.js b/client/homebrew/editor/metadataEditor/validations.js
index 0023c6008..3c35c8a38 100644
--- a/client/homebrew/editor/metadataEditor/validations.js
+++ b/client/homebrew/editor/metadataEditor/validations.js
@@ -1,12 +1,12 @@
 module.exports = {
 	title : [
 		(value)=>{
-			return value?.length > 10 ? 'Max URL length of 10 characters' : null;
+			return value?.length > 10 ? 'Max title length of 10 characters' : null;
 		}
 	],
 	description : [
 		(value)=>{
-			return value?.length > 10 ? 'Max URL length of 10 characters' : null;
+			return value?.length > 10 ? 'Max description length of 10 characters' : null;
 		}
 	],
 	thumbnail : [

From b6c09683beaa8aa7d1297a4590f35809bfe5e080 Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Sat, 5 Nov 2022 19:11:36 -0500
Subject: [PATCH 08/13] add language field validation (for future pr)

fix an oversight in validations
---
 client/homebrew/editor/metadataEditor/validations.js | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/validations.js b/client/homebrew/editor/metadataEditor/validations.js
index 3c35c8a38..8c4f27382 100644
--- a/client/homebrew/editor/metadataEditor/validations.js
+++ b/client/homebrew/editor/metadataEditor/validations.js
@@ -1,17 +1,22 @@
 module.exports = {
 	title : [
 		(value)=>{
-			return value?.length > 10 ? 'Max title length of 10 characters' : null;
+			return value?.length > 100 ? 'Max title length of 100 characters' : null;
 		}
 	],
 	description : [
 		(value)=>{
-			return value?.length > 10 ? 'Max description length of 10 characters' : null;
+			return value?.length > 500 ? 'Max description length of 500 characters.' : null;
 		}
 	],
 	thumbnail : [
 		(value)=>{
-			return value?.length > 5 ? 'Max URL length of 5 characters' : null;
+			return value?.length > 256 ? 'Max URL length of 256 characters.' : null;
+		}
+	],
+	language : [
+		(value)=>{
+			return new RegExp(/[a-z]{2,3}(-.*)?/).test(value || '') === false ? 'Invalid language code.' : null;
 		}
 	]
 };

From 970d03a5e42d07519789b24e6f161491d5253d36 Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Sun, 6 Nov 2022 13:17:39 -0600
Subject: [PATCH 09/13] validate thumbnail is URL

---
 client/homebrew/editor/metadataEditor/metadataEditor.jsx | 2 +-
 client/homebrew/editor/metadataEditor/validations.js     | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index 47110c097..d98a15113 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -275,7 +275,7 @@ const MetadataEditor = createClass({
 						<label>thumbnail</label>
 						<input type='text'
 							defaultValue={this.props.metadata.thumbnail}
-							placeholder='my.thumbnail.url'
+							placeholder='https://my.thumbnail.url'
 							className='value'
 							onChange={(e)=>this.handleFieldChange('thumbnail', e)} />
 						<button className='display' onClick={this.toggleThumbnailDisplay}>
diff --git a/client/homebrew/editor/metadataEditor/validations.js b/client/homebrew/editor/metadataEditor/validations.js
index 8c4f27382..ca7504e27 100644
--- a/client/homebrew/editor/metadataEditor/validations.js
+++ b/client/homebrew/editor/metadataEditor/validations.js
@@ -12,6 +12,14 @@ module.exports = {
 	thumbnail : [
 		(value)=>{
 			return value?.length > 256 ? 'Max URL length of 256 characters.' : null;
+		},
+		(value)=>{
+			try {
+				Boolean(new URL(value));
+				return null;
+			} catch (e) {
+				return 'Must be a valid URL';
+			}
 		}
 	],
 	language : [

From 4f762b376f80304a0596708165ab92ec9843b93c Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Sun, 13 Nov 2022 12:45:31 -0600
Subject: [PATCH 10/13] tweak validation rules on Thumbnail & prevent invalid
 field from block valid fields

---
 client/homebrew/editor/metadataEditor/metadataEditor.jsx | 2 +-
 client/homebrew/editor/metadataEditor/validations.js     | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index d98a15113..9073106e8 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -65,7 +65,7 @@ const MetadataEditor = createClass({
 				[name] : validationErr.length > 0 ? validationErr : undefined
 			}
 		}), ()=>{
-			if(Object.values(this.state.errs ?? {}).filter(Boolean).length === 0 || !this.state.errs[name]){
+			if(validationErr.length === 0){
 				e.target.setCustomValidity('');
 				this.props.onChange({
 					...this.props.metadata,
diff --git a/client/homebrew/editor/metadataEditor/validations.js b/client/homebrew/editor/metadataEditor/validations.js
index ca7504e27..22f9e918b 100644
--- a/client/homebrew/editor/metadataEditor/validations.js
+++ b/client/homebrew/editor/metadataEditor/validations.js
@@ -14,6 +14,7 @@ module.exports = {
 			return value?.length > 256 ? 'Max URL length of 256 characters.' : null;
 		},
 		(value)=>{
+			if(value?.length == 0){return null;}
 			try {
 				Boolean(new URL(value));
 				return null;

From 6a0d8d13b02e6cc0da4badfc6d9cd1568be25984 Mon Sep 17 00:00:00 2001
From: Gazook89 <58999374+Gazook89@users.noreply.github.com>
Date: Sun, 13 Nov 2022 12:56:17 -0600
Subject: [PATCH 11/13] don't save errs to state

---
 .../editor/metadataEditor/metadataEditor.jsx  | 39 +++++++++----------
 1 file changed, 18 insertions(+), 21 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index 9073106e8..d4585ba3f 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -57,30 +57,27 @@ const MetadataEditor = createClass({
 
 	handleFieldChange : function(name, e){
 		e.persist();
+
+		// load validation rules, and check input value against them
 		const inputRules = validations[name] ?? [];
 		const validationErr = inputRules.map((rule)=>rule(e.target.value)).filter(Boolean);
-		this.setState((prevState)=>({
-		    errs : {
-				...(prevState.errs ?? {}),
-				[name] : validationErr.length > 0 ? validationErr : undefined
-			}
-		}), ()=>{
-			if(validationErr.length === 0){
-				e.target.setCustomValidity('');
-				this.props.onChange({
-					...this.props.metadata,
-					[name] : e.target.value
-				});
-			} else {
-				const errMessage = this.state.errs[name].map((err)=>{
-					return `- ${err}`;
-				}).join('\n');
-				e.target.setCustomValidity(errMessage);
-				e.target.reportValidity();
-			};
-		});
-
 
+		// if no validation rules, save to props
+		if(validationErr.length === 0){
+			e.target.setCustomValidity('');
+			this.props.onChange({
+				...this.props.metadata,
+				[name] : e.target.value
+			});
+		} else {
+			// if validation issues, display built-in browser error popup with each error.
+			console.log(validationErr);
+			const errMessage = validationErr.map((err)=>{
+				return `- ${err}`;
+			}).join('\n');
+			e.target.setCustomValidity(errMessage);
+			e.target.reportValidity();
+		};
 	},
 
 	handleSystem : function(system, e){

From 421ed8f51d8c8dfd26b2d71c545ac01b2440bcfe Mon Sep 17 00:00:00 2001
From: Trevor Buckner <calculuschild@gmail.com>
Date: Thu, 17 Nov 2022 21:53:46 -0500
Subject: [PATCH 12/13] Remove comments

---
 client/homebrew/editor/metadataEditor/metadataEditor.jsx | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index d4585ba3f..2f879f7e3 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -39,8 +39,7 @@ const MetadataEditor = createClass({
 
 	getInitialState : function(){
 		return {
-			showThumbnail : true,
-			errs          : null
+			showThumbnail : true
 		};
 	},
 
@@ -248,11 +247,6 @@ const MetadataEditor = createClass({
 		</div>;
 	},
 
-	// renderErrors : function(){
-	// 	if(Object.values(this.state.errs ?? {}).filter(Boolean).length > 0) return;
-	// 	return <span className='validation-error'></span>
-	// },
-
 	render : function(){
 		return <div className='metadataEditor'>
 			<div className='field title'>

From b23a09b1b8d8a5b4fd9fe8cc2430f4aa9cf74665 Mon Sep 17 00:00:00 2001
From: Trevor Buckner <calculuschild@gmail.com>
Date: Thu, 17 Nov 2022 21:58:24 -0500
Subject: [PATCH 13/13] cleanup

---
 client/homebrew/editor/metadataEditor/metadataEditor.jsx | 2 --
 1 file changed, 2 deletions(-)

diff --git a/client/homebrew/editor/metadataEditor/metadataEditor.jsx b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
index 2f879f7e3..b5f9ddc25 100644
--- a/client/homebrew/editor/metadataEditor/metadataEditor.jsx
+++ b/client/homebrew/editor/metadataEditor/metadataEditor.jsx
@@ -15,8 +15,6 @@ const SYSTEMS = ['5e', '4e', '3.5e', 'Pathfinder'];
 
 const homebreweryThumbnail = require('../../thumbnail.png');
 
-
-
 const MetadataEditor = createClass({
 	displayName     : 'MetadataEditor',
 	getDefaultProps : function() {