Sign upLogin

GroceriStar/pdf-export-component

View on GitHub
Star

Showing 120 of 120 total issues

TODO found
Open

// @TODO tests should be. not like right now
Severity: Minor
Found in src/components/GeneratePDFButton/GeneratePDFButton.js by fixme

TODO found
Open

// @TODO someone want to replace it with smacss
Severity: Minor
Found in src/components/Layouts/LayoutStyles/BulletList.js by fixme

TODO found
Open

// @TODO rename this component name.
Severity: Minor
Found in src/components/DownloadLink/DownloadPDFLink.js by fixme

TODO found
Open

// @TODO this structure works like shit. it's not a good animations there.
Severity: Minor
Found in src/components/DownloadLink/DownloadPDFLink.js by fixme

TODO found
Open

// @TODO so we have 3 cases together, but this case is very very different?
Severity: Minor
Found in src/components/Layouts/StandaloneRecipeView/StandaloneRecipeView.js by fixme

TODO found
Open

// @TODO tests
Severity: Minor
Found in src/components/Layouts/LayoutItem/LayoutItem.js by fixme

TODO found
Open

// @TODO Should it be disabled completely? We usually use GeneratePDFButton...
Severity: Minor
Found in src/index.js by fixme

TODO found
Open

// @TODO should be have so long component name here?
Severity: Minor
Found in src/components/DownloadLink/DownloadPDFLink.js by fixme

TODO found
Open

// @TODO replace uuidv1 later
Severity: Minor
Found in src/components/Layouts/LayoutItem/DepartmentLists.js by fixme

TODO found
Open

// @TODO I don't like the name MainLayout. should be more explanatory
Severity: Minor
Found in src/components/DocumentLayouts.js by fixme

TODO found
Open

// @TODO I want to have a cool animation. So when we click on button Generate PDF File - can we also be able to change the button title?
Severity: Minor
Found in src/components/GeneratePDFButton/GeneratePDFButton.js by fixme

TODO found
Open

  // @TODO we should have a flag that will give us 2 options - generate an id by uuid or use for example item.id for it.
Severity: Minor
Found in src/components/Layouts/MainLayout.js by fixme

TODO found
Open

// @TODO I don't like name DocumentLayouts. Looks like this layout is main and it's like a wrapper for inner data
Severity: Minor
Found in src/components/DocumentLayouts.js by fixme

TODO found
Open

  // @TODO i think we can/should pass departments/ingredients together.
Severity: Minor
Found in src/components/Layouts/MainLayout.js by fixme

TODO found
Open

// @TODO I add here everything, but i'm not sure if we need all of that plugins.
Severity: Minor
Found in babel.config.js by fixme

TODO found
Open

// @TODO we need to make a decision. We exporting only one component that will be an entering point to our module
Severity: Minor
Found in src/index.js by fixme

TODO found
Open

// @TODO tests
Severity: Minor
Found in src/components/Layouts/LayoutItem/DepartmentLists.js by fixme

TODO found
Open

// @TODO i don't like this.state.isHidden logic. can be done in more elegant way
Severity: Minor
Found in src/components/GeneratePDFButton/GeneratePDFButton.js by fixme

TODO found
Open

// @TODO this is a bad situation where we have a passing data as it is - we use names, not id at not a main container - component
Severity: Minor
Found in src/components/Layouts/MainLayout.js by fixme

TODO found
Open

// @TODO tests
Severity: Minor
Found in src/components/Layouts/MainLayout.js by fixme
Severity
Category
Status
Source
Language