search

MLH-Fellowship / prep-project-4.1.1

MLH Prep Project for Pod 4.1.1
https://mlh-prep-4-1-1.netlify.app/
MIT License
6 stars 13 forks source link

Shifting the entire logic from App.js to a separate component #21

Open aniketbiswas21 opened 2 years ago

aniketbiswas21 commented 2 years ago

Currently, the fetching, setting of the city, etc. is being done in App.js. Instead, what we can do is create a separate component for this and then import this in App.js. This would also allow me and @Aryaman1706(ISSUE #5 ) to go ahead with using React Context for making the city data available globally instead of prop-drilling.

If this is fine, I can go ahead and make a pr for this change.

CC: @khattakdev @wrussell1999

khattakdev commented 2 years ago

@aniketbiswas21 Sure, go ahead