next/dynamic Error Message Tweaks (vercel#2798)

IanMitchell · arunoda · commit 72edd591e6e3 · 2017-08-17T21:41:52.000+05:30
diff --git a/lib/dynamic.js b/lib/dynamic.js
@@ -13,12 +13,12 @@ export default function dynamicComponent (p, o) {
   } else {
     // Now we are trying to use the modules and render fields in options to load modules.
     if (!p.modules || !p.render) {
-      const errorMessage = 'Options to `next/dynamic` should contains `modules` and `render` fields.'
+      const errorMessage = '`next/dynamic` options should contain `modules` and `render` fields'
       throw new Error(errorMessage)
     }
 
     if (o) {
-      const errorMessage = 'Include options in the first argument which contains `modules` and `render` fields.'
+      const errorMessage = 'Add additional `next/dynamic` options to the first argument containing the `modules` and `render` fields'
       throw new Error(errorMessage)
     }
 

Original file line number	Diff line number	Diff line change
`@@ -13,12 +13,12 @@ export default function dynamicComponent (p, o) {`
`13`	`13`	`} else {`
`14`	`14`	`// Now we are trying to use the modules and render fields in options to load modules.`
`15`	`15`	`if (!p.modules \|\| !p.render) {`
`16`		- const errorMessage = 'Options to `next/dynamic` should contains `modules` and `render` fields.'
	`16`	+ const errorMessage = '`next/dynamic` options should contain `modules` and `render` fields'
`17`	`17`	`throw new Error(errorMessage)`
`18`	`18`	`}`
`19`	`19`
`20`	`20`	`if (o) {`
`21`		- const errorMessage = 'Include options in the first argument which contains `modules` and `render` fields.'
	`21`	+ const errorMessage = 'Add additional `next/dynamic` options to the first argument containing the `modules` and `render` fields'
`22`	`22`	`throw new Error(errorMessage)`
`23`	`23`	`}`
`24`	`24`