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

Add an endpoint to show specific verses #268

Merged

Conversation

Gauravjeetsingh
Copy link
Collaborator

@Gauravjeetsingh Gauravjeetsingh commented Jun 5, 2023

In order to implement the semantic search API, we needed an endpoint that returns all the banidb details of specific verses.

This PR adds endpoint /search-results which take comma separated verse ids as parameter, VerseID and returns all the information required to show those verses as search results.

Data at endpoint /v2/search-results/212,1231
Screenshot from 2023-06-05 22-39-44

Sidenote: Also, pulled the commits from master branch here, so that dev is upto date with master branch commits.

tsingh777 and others added 8 commits April 20, 2021 20:11
* Add ssl cert config for skysql

* Fix linting error and import fs module

* convert serverCert to string from an array

* Added ability to use cluster pools or not (KhalisFoundation#256)

* Added ability to use cluster pools or not (checks if DB_NODES array exists)
Also added ability to optionally use SSL_CA cert for connection

* Added ability to use cluster pools or not (checks if DB_NODES array exists)
Also added ability to optionally use SSL_CA cert for connection

Co-authored-by: Gauravjeet Singh <gaurav.ishwerdas@gmail.com>
…ction-pool-crash

Hotfix: change connection end to release (KhalisFoundation#260)
api/controllers/shabads.js Show resolved Hide resolved
api/controllers/shabads.js Show resolved Hide resolved
@Gauravjeetsingh Gauravjeetsingh merged commit b5b21e9 into KhalisFoundation:dev Jun 22, 2023
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.

6 participants