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

[Finishes #157327287, #156347085] Add search results interface #41

Merged
merged 14 commits into from
Jun 8, 2018

Conversation

adccb
Copy link
Contributor

@adccb adccb commented May 9, 2018

Here we add and wire up the search results page. The search function gets modified some to match the needs we have for the search result data.

Otherwise, the majority of the code involves delinting, tighter type checking, and minor bug fixes.

@adccb adccb requested a review from savannidgerinel May 9, 2018 16:32
Savanni D'Gerinel added 6 commits May 30, 2018 13:38
I switch the data type for dates over to the moment library in order to
better handle date formatting, disable portions that cannot work right
yet, and get the abstract working reasonably.
@savannidgerinel savannidgerinel changed the title added search-results [WIP] added search-results Jun 1, 2018
@savannidgerinel savannidgerinel changed the title [WIP] added search-results Add search results interface Jun 8, 2018
@savannidgerinel savannidgerinel changed the title Add search results interface [Finishes #157327287] Add search results interface Jun 8, 2018
@savannidgerinel savannidgerinel changed the title [Finishes #157327287] Add search results interface [Finishes #157327287, #156347085] Add search results interface Jun 8, 2018
@savannidgerinel savannidgerinel merged commit 0ffac28 into master Jun 8, 2018
@savannidgerinel savannidgerinel deleted the search-results branch June 8, 2018 22:00
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.

None yet

2 participants