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

Recruitment Task - Jakub Pawelski #16

Open
wants to merge 25 commits into
base: main
Choose a base branch
from

Conversation

DragoBJJ
Copy link

@DragoBJJ DragoBJJ commented Jan 8, 2024

Hi everyone,

a small update from my side:

  • I have created a small library of dynamic components in tailwind ( in the future you can make a separate package from it and it can be used for other projects)
  • Atomic Design approach
  • I implemented storyBook so that components can be previewed.
  • I tried to keep the principle of single responsibility.
  • I created a Special hook to paginate pages in case the number of pages would be really large (usePagination)

I would like to hear your opinion on what can be improved :)

Have a productive day !
Greetings Jakub !

@DragoBJJ DragoBJJ changed the title Created members endpoint recruitment task - Jakub Pawelski Jan 8, 2024
@DragoBJJ DragoBJJ changed the title recruitment task - Jakub Pawelski Recruitment Task - Jakub Pawelski Jan 8, 2024
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