Skip to content
This repository has been archived by the owner on Jul 14, 2022. It is now read-only.

[151] Hide filters, add home page collection upon empty search #165

Merged
merged 2 commits into from
Dec 5, 2018
Merged

[151] Hide filters, add home page collection upon empty search #165

merged 2 commits into from
Dec 5, 2018

Conversation

piotrgrundas
Copy link
Contributor

@piotrgrundas piotrgrundas commented Dec 5, 2018 •
edited
Loading

Fixes #151 + implemented this in categories.

Screenshots

image
image

image
image

Pull Request Checklist

  1. All visible strings are translated with proper context.
  2. All data-formatting is locale-aware (dates, numbers, and so on).
  3. The changes are tested.
  4. The code is documented (docstrings, project documentation).

dominik-zeglen
dominik-zeglen suggested changes Dec 5, 2018
View reviewed changes
src/components/HomePage/index.tsx Outdated Show resolved Hide resolved
src/components/ProductsFeatured/index.tsx Outdated Show resolved Hide resolved
src/components/ProductsFeatured/index.tsx Outdated Show resolved Hide resolved
src/components/ProductsFeatured/index.tsx Outdated Show resolved Hide resolved
src/views/Category/CategoryPage.tsx Outdated Show resolved Hide resolved
src/views/Search/index.tsx Outdated Show resolved Hide resolved
tslint.json Outdated Show resolved Hide resolved
@piotrgrundas piotrgrundas force-pushed the 151/empty_search_hide_filters branch 3 times, most recently from 50919a8 to 51eaeb9 Compare December 5, 2018 11:59
@maarcingebala maarcingebala merged commit afb118a into master Dec 5, 2018
@maarcingebala maarcingebala deleted the 151/empty_search_hide_filters branch December 5, 2018 15:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@patrys patrys patrys left review comments

@weronka weronka weronka left review comments

@maarcingebala maarcingebala maarcingebala approved these changes

@dominik-zeglen dominik-zeglen dominik-zeglen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@piotrgrundas @patrys @maarcingebala @dominik-zeglen @weronka