simple task

[atherdon]

All is simple:

• review our current codebase
• find out that we have 4 components that doing the same thing
• make one component that will work in our 4 cases
• after publishing on npm - install to react-pdf-sandbox and check is everything working OK
  @hirdbluebird do you want to grab it?

[hirdbluebird]

@atherdon ku,
razbloch menya

[atherdon]

• We're including https://github.com/GroceriStar/pdf-export-component/blob/master/src/index.js#L6 but i don't see this component at https://github.com/GroceriStar/pdf-export-component/tree/master/src/components/DocumentLayouts

• If you're replacing DocumentLayouts with DocumentDefault you need:

  1. Rename https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/DocumentLayouts/DocumentDefault.js#L27
  2. Export https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/DocumentLayouts/DocumentDefault.js#L44
  3. Import & Export https://github.com/GroceriStar/pdf-export-component/blob/master/src/index.js

• We don't have both of these components: https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/DocumentLayouts/DocumentDefault.js#L6

• Wouldn't work https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/DocumentLayouts/DocumentDefault.js#L9

• If DocumentHOC (stupid name btw) is our DocumentLayouts - you need to change path at index.js. And rename the component to at least name of this file.

• I assume will generate an error... https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/Document.HOC.js#L11-L14

• same to previous point https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/Document.HOC.test.js#L6-L9

• typo https://github.com/GroceriStar/pdf-export-component/tree/master/src/components/Ingridient

• will generate an error https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/Ingridient/List.js#L6

• I don't think that we need this complex folder structure https://github.com/GroceriStar/pdf-export-component/tree/master/src/components/DocumentLayouts/StandaloneRecipeView

• ??? https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/DocumentLayouts/StandaloneRecipeView/styles.js

• we cannot have RenderList4 if we renamed folder and didn't have RenderList1,2,3 https://github.com/GroceriStar/pdf-export-component/blob/master/src/components/DocumentLayouts/StandaloneRecipeView/StandaloneRecipeView.js#L6

[atherdon]

Additional feature: can you update our Readme file and change the explanation of what and how we're including our code right now.

[atherdon]

looks like we can debug our module at react-pdf-print repository. I published your latest version and was able to see warnings