Skip to content

Conversation

@rique223
Copy link
Contributor

@rique223 rique223 commented Feb 9, 2022

Proposed changes (including videos or screenshots)

Implemented a filter by purchase type(free or paid) component for the apps screen of the marketplace. Besides that, new entries on the dictionary, fixed some parts of the App type (purchaseType was typed as unknown and price as string), and created some helpers to work alongside the filter. Will be refactoring the categories filter anchor and then will open this PR for reviews.

Demo gif:
purchaseTypeFIlter

Refactored the categories filter anchor from a plain fuselage select to a select button with dynamic colors.
Demo gif:
New categories filter anchor(PR)

Issue(s)

Steps to test or reproduce

Further comments

rique223 and others added 5 commits January 31, 2022 20:00
Changed the design of the category filter anchor from a button with an icon to a Select with a dynamic text inside
Implemented the logic necessary for the purchase type filter to work, inserted a necessary translation in the dictionary and made some changes to the app TS type so that it works as it should.

The App TS type had purchaseType typed as 'unknown[]' and price as 'string'
@rique223 rique223 changed the title Feat/free paid filter [New] Purchase Type Filter for marketplace apps and Categories filter refactoring Feb 9, 2022
@rique223 rique223 marked this pull request as draft February 9, 2022 00:58
I've refactored the categories filter Anchor from a select dropdown with boolean placeholders to a Button dropdown with improved looks and a selected categories counter.
…gn specs

I've refactored the way the category filter anchor reacts to category selections. Now when no categories are selected the filter looks like a common fuselage select, but when there are selected categories it looks like a blue button with a white counter on it. Also solved some type errors.
@rique223 rique223 changed the title [New] Purchase Type Filter for marketplace apps and Categories filter refactoring [IMPROVE] Purchase Type Filter for marketplace apps and Categories filter refactoring Feb 10, 2022
@rique223 rique223 changed the title [IMPROVE] Purchase Type Filter for marketplace apps and Categories filter refactoring [IMPROVE] Purchase Type Filter for marketplace apps and Categories filter anchor refactoring Feb 10, 2022
@rique223 rique223 requested a review from a team February 10, 2022 18:28
@rique223 rique223 marked this pull request as ready for review February 10, 2022 18:28
@rique223 rique223 changed the title [IMPROVE] Purchase Type Filter for marketplace apps and Categories filter anchor refactoring [NEW] [IMPROVE] Purchase Type Filter for marketplace apps and Categories filter anchor refactoring Feb 11, 2022
@rique223 rique223 requested a review from ujorgeleite February 14, 2022 20:44
Copy link
Contributor Author

@rique223 rique223 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solved reviews

@rique223 rique223 requested review from a team and ujorgeleite February 15, 2022 20:54
@rique223 rique223 merged commit c2202cb into develop Feb 21, 2022
@rique223 rique223 deleted the feat/free-paid-filter branch February 21, 2022 18:26
@pierre-lehnen-rc pierre-lehnen-rc changed the title [NEW] [IMPROVE] Purchase Type Filter for marketplace apps and Categories filter anchor refactoring [IMPROVE] Purchase Type Filter for marketplace apps and Categories filter anchor refactoring Feb 22, 2022
@pierre-lehnen-rc pierre-lehnen-rc mentioned this pull request Mar 1, 2022
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.

4 participants