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

Author autocomplete (front-end) #3414

Merged
merged 1 commit into from
Apr 7, 2022
Merged

Conversation

RC-Lee
Copy link
Contributor

@RC-Lee RC-Lee commented Apr 6, 2022

Issue This PR Addresses

fixes #3226
fixes #1260

Type of Change

Implement author autocomplete on the search bar

Description

Steps to test the PR

cp config/env.production .env
pnpm dev

Please note
It won't work on staging since Parser on staging is messing up with the autocomplete index and the results.
However, testing on cp config/env.staging .env on this implementation won't break search. The autocomplete suggestions simply won't show up.

I did the best I could with styles, but I think getting the implementation in for this PR is more important.
If anyone more experienced with styling would like to tackle this as another issue, it would be much appreciated.

image
image

@gitpod-io
Copy link

gitpod-io bot commented Apr 6, 2022

Copy link
Contributor

@DukeManh DukeManh left a comment

Choose a reason for hiding this comment

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

Can't wait to get this feature in. I left some comments, let me know if you need help with React, I'll be happy to push some changes to this PR.

Copy link
Contributor

@DukeManh DukeManh left a comment

Choose a reason for hiding this comment

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

lgtm

src/web/app/src/components/SearchInput/AuthorInput.tsx Outdated Show resolved Hide resolved
@Kevan-Y
Copy link
Contributor

Kevan-Y commented Apr 7, 2022

@rclee91 Some CSS issues, can be fixed in a follow-up issue.
image

@RC-Lee RC-Lee merged commit 65c2e36 into Seneca-CDOT:master Apr 7, 2022
@RC-Lee RC-Lee deleted the issue#3226 branch April 7, 2022 20:50
@AmasiaNalbandian
Copy link
Contributor

I think this PR caused the following :
image

I opened a new issue about this:
#3426

@AmasiaNalbandian
Copy link
Contributor

@rclee91 Some CSS issues, can be fixed in a follow-up issue. image

Opened issue here: #3427

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Search bar update for Autocomplete with Authors Add autocomplete to search for Authors
5 participants