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

Гранулярные подсказки #17

Merged
merged 10 commits into from
Jan 27, 2024

Conversation

TiGR
Copy link
Contributor

@TiGR TiGR commented Jan 8, 2024

dadata поддерживает гранулярные подсказки, то есть, можно указать какого именно типа географические данные мы хотим получить. С помощью этих параметров (from_bound/to_bound) можно ограничить область подсказок только городами и получить поле, в котором можно указать только город, а не весь адрес. Этот мердж содержит данный функционал - в параметры компонента добавлены поля fromBound/toBound и они соответственно добавляются в тело запроса.

Также, местами наблюдаются конфликты по форматированию - с одной стороны, присутствует .editorconfig с описанными там правилами форматирования, а с другой - сами файлы отформатированы в 4 пробелами (вместо двух) для таба и регулярно встречаются строки с висячими пробелами. Что делать с подобными вещами? Переформатировать приводя к общему виду по мере правки или как? В данном мердже постарался свести к минимуму правки по форматированию, но совсем без них не обошлось

@Stanislavsky69
Copy link
Owner

Stanislavsky69 commented Jan 8, 2024

Здравствуйте! Спасибо за PR. Чтобы было гибко, можно сделать возможность проброса любых параметров и не только bounds. Props назвать locationParameters и спредом в data. editorconfig не принципиален, ставьте как удобно в коде

@TiGR
Copy link
Contributor Author

TiGR commented Jan 10, 2024

Сделал. Но назвал параметр restrictions, потому что там получается как-то очень многословно:

<DaDataNext :location-parameters="{locations_geo: [{lat: ..., lon: ...}], locations: [{...}]}"/>

Получается многократно location там встречается. restrictions - это не принципиально, можно поменять.

@TiGR
Copy link
Contributor Author

TiGR commented Jan 15, 2024

Что-то ещё нужно доделать?

@Stanislavsky69
Copy link
Owner

Что-то ещё нужно доделать?

Два коммента и всё

@TiGR
Copy link
Contributor Author

TiGR commented Jan 15, 2024

Я не вижу никаких комментариев. О чём речь?

@Stanislavsky69
Copy link
Owner

image

@Stanislavsky69
Copy link
Owner

Вам не видно этих комментариев?

@TiGR
Copy link
Contributor Author

TiGR commented Jan 16, 2024

У вас на скринах рядом с комментами написано Pending, то есть они не опубликованы. Опубликуйте, тогда всё сделаем

src/components/useDaData.ts Outdated Show resolved Hide resolved
src/types.ts Outdated Show resolved Hide resolved
@Stanislavsky69
Copy link
Owner

У вас на скринах рядом с комментами написано Pending, то есть они не опубликованы. Опубликуйте, тогда всё сделаем
Отправил, спасибо! В гитлабе с этим попроще, конечно )

@TiGR
Copy link
Contributor Author

TiGR commented Jan 16, 2024

Добавил неймспейс, переместил from_bound и to_bound в отдельные пропсы

@Stanislavsky69
Copy link
Owner

Добавил неймспейс, переместил from_bound и to_bound в отдельные пропсы

Я не правильно выразился ) Имелось в виду не буквально namespace ) А префикс DaData, например, DaDataDivisions.

src/components/useDaData.ts Outdated Show resolved Hide resolved
src/components/useDaData.ts Outdated Show resolved Hide resolved
src/components/useDaData.ts Outdated Show resolved Hide resolved
@Stanislavsky69 Stanislavsky69 added the enhancement New feature or request label Jan 24, 2024
@Stanislavsky69
Copy link
Owner

Обновите еще доку, пожалуйста

@TiGR
Copy link
Contributor Author

TiGR commented Jan 25, 2024

Сделал всё, добавил документацию

@Stanislavsky69 Stanislavsky69 merged commit 29a256f into Stanislavsky69:master Jan 27, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants