Skip to content

React API - #3

Open
Seala11 wants to merge 67 commits into
mainfrom
react-api
Open

React API#3
Seala11 wants to merge 67 commits into
mainfrom
react-api

Conversation

@Seala11

@Seala11 Seala11 commented Dec 7, 2022

Copy link
Copy Markdown
Owner
  1. Task: https://github.com/rolling-scopes-school/tasks/tree/master/react/modules/module03

  2. Screenshot:
    FireShot Capture 188 - React API - react-api-seala11 netlify app

  3. Deploy: https://react-api-seala11.netlify.app/

  4. Done 17.10.2022 / deadline 17.10.2022

  5. Score: 15 / 15

  • App + API – 12 points

    • Create a separate branch.
    • All the logical parts must be in separated components
    • Search bar
    • Rewrite the search page, and split it into 2 logical parts:
    • Search bar. Enter test -> Press Enter -> Send the request to API with the searching parameters -> the list of results is updating
    • The result list of searching: display the data that the API query returned using Cards.
    • Cards
    • every element of the list has to show a small piece of information
    • After the click on the element we have to show all the available information about the element in the modal window.
    • The modal window should be closed by clicking on the cross button in the upper right corner or by clicking on a page outside the modal window
    • When we open a modal window the page should be covered with overlay.
    • Creat a download indicator ( a component with animation, or at least with the string - ‘Downloading’)
  • Tests – 3 points

    • All the api-calls should be mocked in the tests.
    • Coverage 98%
      image
  • Additionally

    • Fully responsive
    • Clear input button
    • Search button
    • Functionality from module 1 of week 1 is preserved ( Input value is stored in LocalStorage during component’s unmount. On initialization, the value is picked from LocalStorage and displayed)

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.

1 participant