DEV: исправить поиск участников по имени и фамилии - #754
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Поиск участников на DEV и PROD зависел от регистра кириллицы, а полное имя сопоставляло отдельные слова через OR: «Иван Иванов» находил также Ивана Петрова и Петра Иванова. Исправлен общий
UserFilter.fullname, используемый публичным Angular-каталогом и административным списком пользователей.Теперь имя и фамилия ищутся совместно, без зависимости от регистра; поддержаны фрагменты, прямой/обратный порядок, составные имена и лишние пробелы. Ввод экранируется как буквальный текст, невозможная по размеру полей строка сразу даёт пустую выдачу. Фильтрация остаётся в SQL до пагинации.
Base DEV:
da29b7b3b4f1ab09f51e3a23cdc40c7a7314a7f6Head:
98e4e3bcf7149be1b0dc5d03840fd8f894a2b534Проверки на PostgreSQL 18 / Python 3.11:
--keepdb;makemigrations --check --dry-run: No changes detected;git diff --check: GREEN.Первый full run без
--keepdbвыполнил все тогдашние 886 тестов, но завершился с exit 1 при удалении тестовой БД из-за оставшихся соединений. Это отражено в документации; runner и тестовая инфраструктура не менялись.Read-only проверка до исправления: «Иван»/«иван»/«ИВАН» дали count 5/0/0 на DEV и 365/10/0 на PROD. Ответы HTTP 200. Точная collation live-БД не читалась; локально крайний случай покрыт тестом с C collation. Авторизованной live-браузерной сессии нет. После исправления live-проверка не заявляется: deploy не выполнялся.
Контракты, permissions, остальные фильтры, React-домен, статистика,
current_application, зависимости и workflows не изменены. Миграций нет. Дополнение: Angular #377 — карточки, debounce и пагинация поиска.Документация: docs/member-name-search.md.
Только Draft → dev. Merge/deploy не выполнять.
GitHub CI: Backend PostgreSQL CI run 36025523510 завершился ошибкой до запуска тестов, на Install dependencies. Poetry/pip:
TypeError: expected string or bytes-like object, got NoneTypeпри чтении версии setuptools, затем ошибка установки idna. Cleanup также не смог импортировать ещё не установленный psycopg2. Lint и обычный CI/Tests прошли. Это не зелёный remote PostgreSQL CI; локальные 887/887 указаны отдельно. Зависимости, lockfile и workflow не менялись, перезапуск вручную не выполнялся.