Skip to content

feat(results list): connected drugs drawer to results list

Tadeusz Miesiąc requested to merge feature/MIN-94-display-hits-for-drugs into development

Description

In this PR I connected first step of the results (groupedSearchResults) and drugs drawer with displaying search results.

Things done

  • Moved components around to reflect stepper. Now we have structure
Drawer
-SearchDrawerWrapper  // controls the displayed component for selected step
--GroupedSearchResults // first step with drawers
--ResultsList // displays list results 2nd step
-Other Drawers...
  • encapsulated redux actions for search drawer steps
  • created pin list component with styling for 4 types of data
  • added fixtures for drawer store since it's commonly used in test scenarios
  • removed dynamic load for search content - it was only ~6kb and caused delay in UX. I would suggest in the future to load the full drawer after app starts. That might be improvement without affecting ux

Things we should consider in the future

It probably would be best idea to add typing for all different possibilities that might occur for each step - this way we will be 100% sure that nothing bads happens. Since there will be changes soon to the stepper (new designs) I decided to stop grinding in place and do it in future update

Merge request reports