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

feat(challenge): coding challenge impl #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

eshalkowski
Copy link
Owner

Client

  • Implemented card chip filters with the ability to have multiple filters on at the same time.
  • created new FilterChipComponent with support for Text, number, and select inputs
  • added loading, and error states to the grid
  • created file.utilities for keeping file related functions abstracted and reusable

Server

  • Fixed bug in server when gt and lt values were not matching properly
  • added error checking when gt and lt are passed

@eshalkowski eshalkowski self-assigned this Oct 10, 2021
@eshalkowski
Copy link
Owner Author

@asibilia - Thank you for taking the time! This won't let me add you as a reviewer until you are listed as a contributor on the project! I can update once the collaboration request is complete.

@eshalkowski eshalkowski requested a review from asibilia October 10, 2021 17:23
@github-pages github-pages bot temporarily deployed to github-pages October 21, 2021 00:52 Inactive
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