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

fix: fixed search #242

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

fix: fixed search #242

wants to merge 8 commits into from

Conversation

NanoNish
Copy link

@NanoNish NanoNish commented Dec 22, 2022

  • changed query to autocomplete type
  • added a path var for defining the field to search in
  • added filter for path var so only certain fields can be filled in
  • updated swagger to include path in search docs

- changed query to autocomplete type
- added a path var for defining the field to search in
- added temp tests for search and checking connection with db
- updated swagger to include path in search docs
- added filter for path var
@NanoNish NanoNish changed the title bfix: fixed search fix: fixed search Dec 24, 2022
- added IsPathValid to constants.go
- removed logic from search.go in controllers
@NanoNish NanoNish self-assigned this Dec 24, 2022
- added IsPathValid to constants.go
- removed logic from search.go in controllers
@NanoNish NanoNish requested a review from renauga December 24, 2022 03:38
@NanoNish
Copy link
Author

IMPORTANT:
Don't push this to prod as Search Index isn't updated in MongoDB

//Checking for possible fields, default is "username"
var possible_path []string
var good_path bool = false
possible_path = []string{"username", "fullname", "handle.codechef", "handle.codeforces", "handle.hackerearth", "handle.hackerrank", "handle.spoj"}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Re-check the list of platforms. Leetcode is missing and hackerearth is extra.

@NanoNish NanoNish requested a review from renauga February 19, 2023 19:26
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.

2 participants