Skip to content
This repository was archived by the owner on Sep 13, 2022. It is now read-only.

Comments

Mohamnmed Chowdhury | Instructor : Edgardo Molina | TA : Silu#105

Open
moiscoding wants to merge 3 commits intoCUNYTechPrep:masterfrom
moiscoding:master
Open

Mohamnmed Chowdhury | Instructor : Edgardo Molina | TA : Silu#105
moiscoding wants to merge 3 commits intoCUNYTechPrep:masterfrom
moiscoding:master

Conversation

@moiscoding
Copy link

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ Feature
  • 🐛 Bug Fix
  • 👷 Optimization
  • 📝 Documentation Update
  • 🔖 Release
  • 🚩 Other

Description

This PR [adds/removes/fixes/replaces] this [feature/bug/etc].

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 readme
  • 📜 contributing.md
  • 📓 docs
  • 📕 storybook
  • 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@moiscoding
Copy link
Author

the functions City and ZipSearchFields are functional components
that are responsible for displaying the city information and the text input field respectively

in the app class, a state object is created that holds the zipcode entered into the text field ,
and a array of cities, which stores the data returned from the api call to
http://ctp-zip-api.herokuapp.com/zip/

the render is responsible for displaying the content on the screen, hence the City and ZipSearchField components were
called inside of the render.

since there can be multiple cities assiciated with a zipcode, each of the elemts in the cities[] array
was mapped to a City component.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant