Skip to content

Instantly share code, notes, and snippets.

@khalidwilliams
Created November 10, 2020 00:05
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save khalidwilliams/ae2ea891ac60b9a10f06f7debff24a32 to your computer and use it in GitHub Desktop.
Save khalidwilliams/ae2ea891ac60b9a10f06f7debff24a32 to your computer and use it in GitHub Desktop.

Final Project Submission

Rocky Balboa victoriously cheering

You did it!! Congratulations on getting through the Mod 3 projects.

There are a lot of y'all, and not a lot of time to give detailed feedback on every piece of all your projects. So please comment down below with

  • the link to your repo
  • the link to your deployment (if you have one, not stress if not)
  • 1 - 2 dpecific areas of the code you want feedback on.

Thank you, and great work!

@hannakim91
Copy link

hannakim91 commented Nov 10, 2020

  • Repo
  • use of hooks - had to use conditional rendering sometimes instead of creating more hooks - not sure which is better?
  • how GET requests are formatted for comparing cities

@MichaelEWalker87
Copy link

  • Don't Fear the Repo
  • How to refactor the last test in App (tried to use map but ran into a ton of errors)
  • Data flow

@holladayian
Copy link

link

  • test layouts (is there too much in each it block?)
  • Are ternaries acceptable in return statements?

@bretm9
Copy link

bretm9 commented Nov 10, 2020

Repo link

Deployed site

  • userEvent and async testing
  • Hoping the use of functional components over class based-based components for everything other than App is acceptable. Specifcally in the case of Search.
  • Also if you have the time, are my font imports the right convention?

@ericberglund117
Copy link

You Repo What You Sow

  • My App integration test. How can I test a user story without repeating a bunch of code? I just feel like my last it block could have been more succinct.

@Atos20
Copy link

Atos20 commented Nov 10, 2020

repo
deployed Site

  • testing using async and UserEvent
  • use of state

@nicolegooden
Copy link

nicolegooden commented Nov 10, 2020

Project repo: https://github.com/nicolegooden/potter
Not yet deployed

  1. Are my test suites thorough enough? Would really appreciate a response to some of my comments in my test files!
  2. Are my functional components too crowded with conditionals?

Thanks in advance!

@Kristiannmiller
Copy link

Kristiannmiller commented Nov 10, 2020

Lyrical Lies Repo
Deployed Page - note that audio doesn't work on deployed site for some reason

  1. Are my tests thorough and in the correct components?
  2. Is the basic architecture a good structure or would you recommend changes? Would you recommend changing the way I use state?

@Chriscastanuela
Copy link

@krogowsk531
Copy link

https://github.com/krogowsk531/under-the-weather

  • Were there components that I may be able to refactor and combine into one?
  • Where I should be more thoroughly testing?

@nware1066
Copy link

Repo: https://github.com/nware1066/pub-menu

sad path testing

@aemcdonald
Copy link

Repo

  • Testing: are my tests thorough enough?

  • Project architecture, could I have been leaner with my components?

@nathanielmillard
Copy link

nathanielmillard commented Nov 10, 2020

Link to Repo
Not Reployed

  • Breakdown of logic between components
  • Best Practices behavior on state/data down actions up

@timkeresey
Copy link

Repo

  • async JS organization and testing

@stacyp2006
Copy link

Repo

Testing: specifically, whether it is thorough enough and why my catches are logging in the terminal.
Architecture: Organization and state management

@GreyMatteOr
Copy link

@BlakeDonn
Copy link

BlakeDonn commented Nov 10, 2020

repo
env api key: REACT_APP_API_KEY=FB1509FF-2D29-7C41-AD87-7E52E19A0A22795D51E2-8C6C-4F81-94A1-20C7415D8B7F

  1. Is this an acceptable use of router? (specifically the non-exact use of results page if too brief)
  2. Data flow

@taylorjohnson141
Copy link

https://github.com/taylorjohnson141/learnlo
Architecture: Is everything set up okay
Testing:Am I testing everything correctly

@BrigetteDoelp
Copy link

https://github.com/BrigetteDoelp/wayshrine-adventurer-registry

TESTING PLEASE
and also architecture! had a second guessing moment there, so i'd like to hear about how my react stuff is plz

@kathrynljackson
Copy link

PokéPlanner

Testing:

  • What did I not test that I should have?
  • How can I improve the asyn testing?

Data:

  • Where the components separated correctly?
  • Where the props passed correctly?
  • Why are the errors in the console mad at me?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment