Handle TODO comments

[Avuidrauxs]

This issue was opened to handle tasks related to all the TODO comments in the project

• App.js, https://github.com/Avuidrauxs/showcase/blob/master/src/App.js#L4-L12
• InsideLayout.js, https://github.com/Avuidrauxs/showcase/blob/master/src/InsideLayout.js#L17
• DepartmentInfo.js, https://github.com/Avuidrauxs/showcase/blob/master/src/DepartmentInfo.js#L12
• DownloadPDFLink.js, https://github.com/Avuidrauxs/showcase/blob/master/src/DownloadPDFLink.js#L16
• RenderLists.js, https://github.com/Avuidrauxs/showcase/blob/master/src/RenderLists.js#L7
• AntdLayout.js, https://github.com/Avuidrauxs/showcase/blob/master/src/AntdLayout.js#L12
• CleanLayout.js, https://github.com/Avuidrauxs/showcase/blob/master/src/CleanLayout.js#L28
• ChangeGroceryNameView.js, https://github.com/Avuidrauxs/showcase/blob/master/src/ChangeGroceryNameView.js#L5
• GroceriesView.js, https://github.com/Avuidrauxs/showcase/blob/master/src/GroceriesView.js#L11
• HomeView.js, https://github.com/Avuidrauxs/showcase/blob/master/src/HomeView.js#L27
• ManageGroceryView.js, https://github.com/Avuidrauxs/showcase/blob/master/src/ManageGroceryView.js#L5

[Avuidrauxs]

@atherdon Hasn't this been achieved already , from the TODO, i'm reading

[atherdon]

@Avuidrauxs can you add a codeline to each of this todos, so i can quickly review it and undestand a context?
like this: https://github.com/GroceriStar/showcase/blob/master/src/App.js#L4-L8

for this case - yeah - just delete a todo comment and next import of homeview.
Note: please fork showcase and apply your changes via pull request

[Avuidrauxs]

@atherdon , okay understood. I have updated the issue description with links now as requested.

[Avuidrauxs]

@atherdon , Can you clarify what you want me do with reworm over here ? https://github.com/GroceriStar/showcase/blob/master/src/App.js#L12,

[atherdon]

@Avuidrauxs let's skip a reworm for now. or spend some time and generate an explanation for me how you'll do it and how it'll benefit to us. because i didn't read a lot about it.

[atherdon]

find some paper notes, related to showcase project- will add them as tasks for future at issues.
tell me if you pick some of this todos or i should pick it for you. just not sure what the status is

[atherdon]

I'm also jumping into this project for a few days, so be ready to my avalanche of small commits :(

[Avuidrauxs]

@atherdon , I'm on them. But I'll be wary of your commits 😄

[Avuidrauxs]

@atherdon , What kinda links method from react-router you want me to use in https://github.com/Avuidrauxs/showcase/blob/master/src/InsideLayout.js#L17,

Do you want me to refactor the function to use <Links> or rather use history.push() from History to redirect to onClick of the button.

[Avuidrauxs]

@atherdon , Also yeah I will see if reworm is still relevant for what we want to achieve.

[atherdon]

@Avuidrauxs we should connect somehow the getLink function with our router component.
I mean - it's bad that we hardcoded this path '/grocery/:id' . let's assume this situation: you change links at router, for example to '/grocerylist/:id' and after this chages - getLink method is actually a bug, and only if you remember about it - you'll fix it.