Commitaa86304

committed

clean up logs

Signed-off-by: shmck <shawn.j.mckay@gmail.com>

1 parentb63c31b commitaa86304Copy full SHA for aa86304

File tree

+25

-13

lines changed

+25

-13

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -329,7 +329,7 @@ class Channel implements Channel {`
`329`	`329`	`case'TEST_PASS':`
`330`	`330`	`consttutorial=this.context.tutorial.get()`
`331`	`331`	`if(!tutorial){`
`332`		`-thrownewError('Error with current tutorial')`
	`332`	`+thrownewError('ERROR: Tutorial not found in test run')`
`333`	`333`	`}`
`334`	`334`	`// update local storage stepProgress`
`335`	`335`	`constprogress=this.context.progress.setStepComplete(tutorial,action.payload.stepId)`
`@@ -340,7 +340,7 @@ class Channel implements Channel {`
`340`	`340`	`// send message`
`341`	`341`	`constsentToClient=awaitthis.postMessage(action)`
`342`	`342`	`if(!sentToClient){`
`343`		-thrownewError(`Message post failure:${JSON.stringify(action)}`)
	`343`	+thrownewError(`ERROR:Message post failure:${JSON.stringify(action)}`)
`344`	`344`	`}`
`345`	`345`	`}`
`346`	`346`	`}`

Lines changed: 11 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,13 +1,19 @@`
`1`	`1`	`import{LOG}from'../../environment'`
`2`	`2`
`3`		`-constlogger=(message:string\|string[])=>{`
	`3`	`+exporttypeLog=string\|object`
	`4`	`+`
	`5`	`+constlogger=(...messages:Log[]):void=>{`
`4`	`6`	`if(!LOG){`
`5`	`7`	`return`
`6`	`8`	`}`
`7`		`-if(Array.isArray(message)){`
`8`		`-message.forEach(console.log)`
`9`		`-}else{`
`10`		`-console.log(message)`
	`9`	`+// Inside vscode, you console.log does not allow more than 1 param`
	`10`	`+// to get around it, we can log with multiple log statements`
	`11`	`+for(constmessageofmessages){`
	`12`	`+if(typeofmessage==='object'){`
	`13`	`+console.log(JSON.stringify(message))`
	`14`	`+}else{`
	`15`	`+console.log(message)`
	`16`	`+}`
`11`	`17`	`}`
`12`	`18`	`}`
`13`	`19`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -10,8 +10,8 @@ class ErrorBoundary extends React.Component {`
`10`	`10`	`// Display fallback UI`
`11`	`11`	`this.setState({errorMessage:error.message})`
`12`	`12`	`// You can also log the error to an error reporting service`
`13`		`-logger('ERROR in component:',JSON.stringify(error))`
`14`		`-logger('ERROR info:',JSON.stringify(info))`
	`13`	`+logger('ERROR in component:',error)`
	`14`	`+logger('ERROR info:',info)`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`publicrender(){`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -59,14 +59,14 @@ const useRouter = (): Output => {`
`59`	`59`	`}elseif(Array.isArray(path)){`
`60`	`60`	`pathMatch=path.some((p)=>state.matches(p))`
`61`	`61`	`}else{`
`62`		-thrownewError(`Invalid route path${JSON.stringify(path)}`)
	`62`	+thrownewError(`ERROR:Invalid route path:${JSON.stringify(path)}`)
`63`	`63`	`}`
`64`	`64`	`if(pathMatch){`
`65`	`65`	`//@ts-ignore`
`66`	`66`	`returnchild.props.children`
`67`	`67`	`}`
`68`	`68`	`}`
`69`		-constmessage=`No Route matches for${JSON.stringify(state)}`
	`69`	+constmessage=`ERROR:No Route matches for${JSON.stringify(state)}`
`70`	`70`	`onError(newError(message))`
`71`	`71`	`console.warn(message)`
`72`	`72`	`returnnull`

Lines changed: 8 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,13 +1,19 @@`
`1`	`1`	`import{LOG}from'../../environment'`
`2`	`2`
`3`		`-constlogger=(...messages:string[])=>{`
	`3`	`+exporttypeLog=string\|object`
	`4`	`+`
	`5`	`+constlogger=(...messages:Log[]):void=>{`
`4`	`6`	`if(!LOG){`
`5`	`7`	`return`
`6`	`8`	`}`
`7`	`9`	`// Inside vscode, you console.log does not allow more than 1 param`
`8`	`10`	`// to get around it, we can log with multiple log statements`
`9`	`11`	`for(constmessageofmessages){`
`10`		`-console.log(message)`
	`12`	`+if(typeofmessage==='object'){`
	`13`	`+console.log(JSON.stringify(message))`
	`14`	`+}else{`
	`15`	`+console.log(message)`
	`16`	`+}`
`11`	`17`	`}`
`12`	`18`	`}`
`13`	`19`

Comments

(0)