Skip to content

Module 02 | React. Routing#2

Open
Quiddlee wants to merge 145 commits intomainfrom
module-02
Open

Module 02 | React. Routing#2
Quiddlee wants to merge 145 commits intomainfrom
module-02

Conversation

@Quiddlee
Copy link
Owner

@Quiddlee Quiddlee commented Oct 30, 2023

  1. Task: click
  2. Screenshot:
    image

image

image

  1. Deploy: click
  2. Done 30.10.2023 / deadline 06.11.2023
  3. Score: 100 / 100

Functional requirements 🍿

  • 🌓 Conversion of class components to functional components with hooks - 10
  • 🔍 Pagination is present in both URL and on the page - 20
  • 📥 User-initiated change of items per page triggers a new request, displaying the first page - 15
  • 🌋 Upon clicking, open details panel on the right side of the page: - 5
  • 📤 use router outlet, left side of the page should continue displaying the list of results - 10
  • 📞 initiate an additional API call, display a loader, and update the URL - 5
  • 🔐 Details panel should be closed either on the "close" button click or on the main panel click - 10

Penalties 👎

  • 😥 TypeScript isn't used: -95 points
  • 😱 Usage of any: -20 points per each
  • 🙀 Usage of ts-ignore: -20 points per each
  • 🧃 Direct DOM manipulations inside the React components: -50 points per each
  • 🪝 React hooks are used to get access to either state, or to the component lifecycle: -70 points
  • 💨 Presence of code-smells (God-object, chunks of duplicate code), commented code sections: -10 points per each
  • 🗿 Usage of Redux or other state management libraries: -100 points
  • 📚 Usage of component libraries, e.g. Material UI, Ant Design: -100 points
  • 📛 Commits after the deadline: -40 points

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.

2 participants