Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fetch /gsoc command data from Algolia instead of DB #712

Merged
merged 11 commits into from
Feb 4, 2025

Conversation

Dishant1804
Copy link
Collaborator

Resolves #686

Made a separate file for gsoc search api that fetches the data from algolia, also the searchable attribute is set to idx_custom_tags so that we only get the tag list, which was expected in the issue.

@Dishant1804
Copy link
Collaborator Author

@arkid15r I've made necessary changes. Please let me know if any changes are required :)

Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

We already have project search related entity, let's use that instead of adding another similar entity.

backend/apps/owasp/api/search/gsoc.py Outdated Show resolved Hide resolved
@github-actions github-actions bot added the schema label Feb 3, 2025
@github-actions github-actions bot removed the schema label Feb 4, 2025
@Dishant1804 Dishant1804 requested a review from arkid15r February 4, 2025 08:43
@arkid15r arkid15r enabled auto-merge February 4, 2025 22:48
Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

LGTM

@arkid15r arkid15r added this pull request to the merge queue Feb 4, 2025
Merged via the queue into OWASP:main with commit 3942435 Feb 4, 2025
13 checks passed
Dishant1804 added a commit to Dishant1804/Nest that referenced this pull request Feb 7, 2025
* gsoc command data fetch from algolia

* pre-commit and format checks

* moved get_gsoc_projects to search project file

* Update code

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>
github-merge-queue bot pushed a commit that referenced this pull request Feb 7, 2025
* anchor tags migrated

* Buttons migrated to chakraUI

* migarted buttons tooltip and anchortags to chakraUI

* fixed focus ring issue

* Generic component for details page (#678)

* base_design

* Add types and apply corrections.

* Improvement

* cleanup

* added issue label

* fixes

* pre-commit

---------

Co-authored-by: Kate Golovanova <kate@kgthreads.com>

* Addition of Contribute Section in Home of Slack NestBot (#593)

* Addition of Contribute Section in Home of Slack NestBot

* Update contribute.py

* Update of apps/slack/commands/contribute

* Minor changes

* Update of code after the review

* Update backend/apps/slack/commands/contribute.py

Co-authored-by: Nitin Awari <99824048+nitinawari@users.noreply.github.com>

* Update urls.py

* Update contribute.py

* Update code

---------

Co-authored-by: Nitin Awari <99824048+nitinawari@users.noreply.github.com>
Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>

* resolved conflicts

* sort dropdown tooltip position modified

* test utils warpper used

* checks and format

* removed unnecessary button props

* removed redundant lines

* Update FE dependencies

* tooltip-bug (#744)

* Addition of website attribute to project schema (#740)

* Addition of website attribute to project schema

* Update vite.config.ts

* Update website-empty.yaml

* Update website-none.yaml

* Update website-none.yaml

* Update website-empty.yaml

* Update website-none.yaml

* Update website-empty.yaml

* Update website-empty.yaml

* Update project_test.py

* Update project.json

* Delete schema/tests/data/project/negative/website-empty.yaml

* Update code

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>

* Add blog to project schema (#745)

* added blog attribute to project schema

* added blog attribute

* Update project_test.py

* Update code

* Update code

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>

* Initialize OWASP chapter schema

* Refactor index synonyms (#748)

* Changes5

* Refactor index synonyms

* Update code

---------

Co-authored-by: Raghav Gupta 2 <raghavgupta1@Raghavs-MacBook-Pro-2.local>
Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>

* write repository schema and tests (#741)

* write repository schema and tests

* fixed repository schema tests

* Update code

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>

* Fetch /gsoc command data from Algolia instead of DB (#712)

* gsoc command data fetch from algolia

* pre-commit and format checks

* moved get_gsoc_projects to search project file

* Update code

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>

* Reconfigure spell check

* Bump @chakra-ui/react from 3.6.0 to 3.7.0 in /frontend (#752)

Bumps [@chakra-ui/react](https://github.com/chakra-ui/chakra-ui/tree/HEAD/packages/react) from 3.6.0 to 3.7.0.
- [Release notes](https://github.com/chakra-ui/chakra-ui/releases)
- [Changelog](https://github.com/chakra-ui/chakra-ui/blob/main/packages/react/CHANGELOG.md)
- [Commits](https://github.com/chakra-ui/chakra-ui/commits/@chakra-ui/react@3.7.0/packages/react)

---
updated-dependencies:
- dependency-name: "@chakra-ui/react"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update params mapping

* Added a method to convert block to text and fixed warning  (#754)

* Added a method to convert block to text

* Added testcase for utils

* Update code

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>

* Update frontend dependencies

* Run poetry update

* conflicts resolved

* Cleanup: removing unused dependencies n dev dependencies (#601)

* "Cleanup: removing unused dependencies n dev dependencies "

* resolved conflicts

* Fix staging image build

* "Cleanup: removing unused dependencies n dev dependencies "

* fix run npm install install prettier plugin tailwindcss

* fix run run npm lint check

* fix check for uncommited changes

* resolved conflicts  n npm format check

* "Cleanup: removing unused dependencies n dev dependencies "

* resolved conflicts

* Fix staging image build

* "Cleanup: removing unused dependencies n dev dependencies "

* fix run npm install install prettier plugin tailwindcss

* fix run run npm lint check

* fix check for uncommited changes

* resolved conflicts  n npm format check

* updated one

* updated fix

* removed unwanted files from updated fix

* Update code

---------

Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Kate Golovanova <kate@kgthreads.com>
Co-authored-by: Raj gupta <134638667+Rajgupta36@users.noreply.github.com>
Co-authored-by: AbhayTopno <143319154+AbhayTopno@users.noreply.github.com>
Co-authored-by: Nitin Awari <99824048+nitinawari@users.noreply.github.com>
Co-authored-by: Arkadii Yakovets <arkadii.yakovets@owasp.org>
Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>
Co-authored-by: yashgoyal0110 <149111979+yashgoyal0110@users.noreply.github.com>
Co-authored-by: Rusheek <64518082+Rusheek@users.noreply.github.com>
Co-authored-by: Raghavg18 <graghav2006@gmail.com>
Co-authored-by: Raghav Gupta 2 <raghavgupta1@Raghavs-MacBook-Pro-2.local>
Co-authored-by: Shyam Kumar <kr.shyamiit@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Abhay <grabhaymishra@gmail.com>
Co-authored-by: Diamond Girl <sneha08115@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor /gsoc command to fetch data from algolia index
2 participants