DEV Community

Discussion on: React.js Interview - technical submission and detailed feedback

Collapse
 
rajeshroyal profile image
Rajesh Royal

⚫ Second Point.
The default exporter of containers is created because there can be multiple exports of containers. [ex. - HOC for error boundary, HOC for Auth Provider and etc.] That's why it have only exports. But in case of a particular page there will be only page itself to be exported.

⚫ Third Point.
I accept it looks a little bit complex for just 2 routes, I could have just added two simple routes. But keeping the scalling of routes in mind I implemented it this way so It will be more easy to add childrens to the routes anytime in future.

Your feedback is really helpfull, what idea I get from the company feedback is that I should be doing what needed and in simply ways, instead of thinking the future prespectives. [deliver what needed 😄]

Thanks @shinspiegel

Collapse
 
shinspiegel profile image
Jeferson 'Shin' Leite Borges

I was afraid that you get very pissed by my comment.
Thanks for understanding. :D

Thread Thread
 
ninjainpajama profile image
ninja in pajama

😅

Thread Thread
 
rajeshroyal profile image
Rajesh Royal • Edited

@shinspiegel hehe.. It looks like that when you start reading from top 🤣🤣

Thread Thread
 
stolengalaxy profile image
Matthew Ridley

not so sure this is right,
unforu
Image description

Thread Thread
 
stolengalaxy profile image
Matthew Ridley • Edited

thank, this is sample of work i done, rate pleaser.

py
Enter fullscreen mode Exit fullscreen mode