MLH-Fellowship / prep-project-22.AUG.PREP.2

MLH Prep Project for Pod 22.JUL.AUG.2
https://prep-22-aug-prep-2-project.netlify.app/
MIT License
3 stars 13 forks source link

Divided the project into more components #12

Closed nightknighto closed 2 years ago

nightknighto commented 2 years ago

Made some early commits so that we can get to work quicker. These are more restructure required. This is just an initial step.

closes #8

netlify[bot] commented 2 years ago

Deploy Preview for prep-22-aug-prep-2-project ready!

Name Link
Latest commit 00e961959fd4d15ac256a5bd9fcb8ec90d6334b5
Latest deploy log https://app.netlify.com/sites/prep-22-aug-prep-2-project/deploys/62f14d9b423a230008b8eca6
Deploy Preview https://deploy-preview-12--prep-22-aug-prep-2-project.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

omeiirr commented 2 years ago

Should the file names have spaces in them? 🤔

omeiirr commented 2 years ago

Also, for the styling, how's this structure?

- components
|  - ErrorComponent
   | - index.js
   | - style.scss   
nightknighto commented 2 years ago

@omeiirr I don't think there is built-in support for scss. As for the structure, sounds elegant and tidy for complex apps, but I don't think we'll need that level of structure, maybe?