Commit3827240

committed

clean up logs

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

1 parent065df38 commit3827240Copy full SHA for 3827240

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
`@@ -277,7 +277,7 @@ class Channel implements Channel {`
`277`	`277`	`case'TEST_PASS':`
`278`	`278`	`consttutorial=this.context.tutorial.get()`
`279`	`279`	`if(!tutorial){`
`280`		`-thrownewError('Error with current tutorial')`
	`280`	`+thrownewError('ERROR: Tutorial not found in test run')`
`281`	`281`	`}`
`282`	`282`	`// update local storage stepProgress`
`283`	`283`	`constprogress=this.context.progress.setStepComplete(tutorial,action.payload.stepId)`
`@@ -288,7 +288,7 @@ class Channel implements Channel {`
`288`	`288`	`// send message`
`289`	`289`	`constsentToClient=awaitthis.postMessage(action)`
`290`	`290`	`if(!sentToClient){`
`291`		-thrownewError(`Message post failure:${JSON.stringify(action)}`)
	`291`	+thrownewError(`ERROR:Message post failure:${JSON.stringify(action)}`)
`292`	`292`	`}`
`293`	`293`	`}`
`294`	`294`	`}`

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)