feature(accordion): Added accordion components to the project

Description

Added accordion components to the project. Decided to use library because of bigger flexibility and time-save. I had two options

• Custom component Pros: Small bundle size, highly customizable Cons: Time consuming: would have to implement reusable and composable component + analise all cases that occurs in the app. There is danger that all cases are not included in designs so it would require to spend even more time
• Library Pros:
• huge time save, customizable Cons: 100kb unpacked size, 10kb gziped -> It would highly increase bundle size. But it's ok for me cuz it's CSR where it doesn't matter that much comparing to SSR or Static website

Implementation reasoning

Imported each of library module and exported it as custom component to add custom styling to it.

Additionally

• Moved global styles import to pages wrapper (_app.tsx)
• Removed redux-page example - its no longer needed
• Created usage example http://localhost:3000/accordion-test

Why the ...rest param is placed before className? IMO it's a good idea to use rest param also in the className (className={twMerge(...)}), this will give us ability to easily modify this field if needed

fixed

Should this type be only local? IMO it's a good idea to create a global ID type if it's consistent in every use case

yes, it's only accordion specific type extracted from node_package

This should not be there as it's a test component.

IMO we should use storybook for that, what do you think?

I think introducing storybook at this stage would be a waste of time. I wanted to let you quickly see usage of the components introduced in PR. I'm going to remove it after finishing https://minerva-appunite.atlassian.net/browse/MIN-65. Is it ok?

actually... going to remove it now

changed this line in version 2 of the diff

LGTM but removing accordion-test.tsx (or replacing with storybook component) is an RFC from my side

approved this merge request

added 1 commit

• 4e3f22d1 - refactor(removed accordion test page): removed page used for displaying test cases

Compare with previous version

enabled an automatic merge when the pipeline for 4e3f22d1 succeeds

merged

mentioned in commit 2d8c3ea6