Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release/1.0.0 #37

Closed
wants to merge 50 commits into from
Closed

Release/1.0.0 #37

wants to merge 50 commits into from

Conversation

Leboroz
Copy link
Owner

@Leboroz Leboroz commented Mar 2, 2022

Hello reviewer, 👋

we did it

Live Version: HERE

Objectives of the project

  • Use React documentation.
  • Use React components.
  • Use React props.
  • Use React Router.
  • Connect React and Redux.
  • Handle events in a React app.
  • Write unit tests with React Testing Library.
  • Use styles in a React app.
  • Use React life cycle methods.
  • Apply React best practices and language style guides in code.
  • Use store, actions, and reducers in React.
  • Perform a code review for a team member.

We used create-react-app and react-redux. We created routes and view components. We used react-router-bootstrap for the page navigation links and style active class to indicate which section/page the user is currently on. Then, we created directories for all Redux state slice files.

Upon first render fetch data from the SpaceX API endpoints:

Once the data are fetched, dispatch an action to store the selected data in Redux store:

  • Rockets:
    • id
    • rocket_name
    • description
    • flickr_images
  • Missions:
    • mission_id
    • mission_name
    • description

render list

we styled the project using React Bootstrap

  • We rendered a list of rockets. For the image of a rocket, we used the first image in the array of flickr_images.
  • Render a table with the missions' data (as per design).
  • We render a table with the missions' data (as per design).

My profile

  • We composed two layouts and listed ONLY the rockets reserved and missions joined by the user (as per design):

Leboroz and others added 30 commits February 28, 2022 12:04
Fetch Data, Display Data, Create redux directory
I created a card component importing Card from React-bootstrap. Then, I
created a list component that maps the fetched API object into card
component.
Add badge object that contains style and text
Copy link

@DeliceLydia DeliceLydia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congratulations! 🎉🎉

Hi Team,

Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

You Rock!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

@Leboroz Leboroz closed this Mar 2, 2022
@Leboroz Leboroz deleted the release/1.0.0 branch March 2, 2022 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants