Skip to content

feat: integrate Meilisearch for enhanced search functionality - #59

Open
fayzetwin1 wants to merge 2 commits into
mainfrom
more-awesome-search
Open

feat: integrate Meilisearch for enhanced search functionality#59
fayzetwin1 wants to merge 2 commits into
mainfrom
more-awesome-search

Conversation

@fayzetwin1

Copy link
Copy Markdown
Contributor

Changelog:

  • Added Meilisearch configuration to .env.example and settings.py.
  • Updated docker-compose files to include Meilisearch service with health checks.
  • Implemented search functionality in UserViewSet and MarketplaceViewSet using Meilisearch.
  • Created search suggestion endpoint and integrated it into the frontend search bar.
  • Enhanced search UI with new styles and tabs for apps and users.
  • Updated Russian translations for new search-related messages.
image image image

- Added Meilisearch configuration to .env.example and settings.py.
- Updated docker-compose files to include Meilisearch service with health checks.
- Implemented search functionality in UserViewSet and MarketplaceViewSet using Meilisearch.
- Created search suggestion endpoint and integrated it into the frontend search bar.
- Enhanced search UI with new styles and tabs for apps and users.
- Updated Russian translations for new search-related messages.
@fayzetwin1 fayzetwin1 self-assigned this Aug 29, 2026

@myslivets myslivets left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

В целом норм, но нужно мне смотреть на демонстрацию фичи, чтобы понять, есть ли ещё огрехи или нет

Comment thread apps/marketplace/urls.py Outdated
Comment thread templates/search.html
<a href="?{% if url_params_tabs %}{{ url_params_tabs|safe }}&amp;{% endif %}type=apps"
class="tab{% if search_type != 'users' %} active{% endif %}"
style="text-decoration:none; color:black; display:inline-block;">
{% trans "PAGE_SEARCH_TAB_APPS" %}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Не по гайдлайнам: отсутствует иконка у вкладки (programs.png)

Comment thread templates/search.html
Comment thread templates/search.html Outdated
Comment thread staticfiles/js/search_suggest.js Outdated
@notsecret808

Copy link
Copy Markdown
Contributor

Я не увидел явных инъекций, но на всякий случай пересмотри самостоятельно места, где ты берешь информацию из query params и передаешь в melisearch.

Comment thread apps/api/v2/views.py Outdated
Comment thread apps/marketplace/views.py Outdated
@notsecret808

Copy link
Copy Markdown
Contributor

LGTM

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.

4 participants