Skip to content

Feat/dev 3958 certificate by phone number#102

Open
gochicus wants to merge 5 commits into
masterfrom
feat/DEV-3958-certificate-by-phone-number
Open

Feat/dev 3958 certificate by phone number#102
gochicus wants to merge 5 commits into
masterfrom
feat/DEV-3958-certificate-by-phone-number

Conversation

@gochicus

@gochicus gochicus commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

DEV-3958

В драфте пока, нужно добавить иллюстрации настроек и их описать

gochicus added 2 commits June 8, 2026 17:39
Signed-off-by: Aleksandr Zavadkin <az@rees46.ru>
Signed-off-by: Aleksandr Zavadkin <az@rees46.ru>
@gochicus gochicus self-assigned this Jun 8, 2026
@gochicus gochicus added the documentation Improvements or additions to documentation label Jun 8, 2026

@gochicus gochicus left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Тут нужно что то

@gochicus gochicus marked this pull request as ready for review June 10, 2026 07:04

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Может коротко сказать что такое персональный сертификат (или дать ссылку на описание)
главное - может быть принят к оплате только при предъявлении номера телефона юзера, на которого оформлен этот сертификат (доп функция проверки)

Ну и допом, может быть использован для оплаты только по предъявлению номера телефона

какая цель статьи - она описывает перс сертификат или логику его применения?
со старта не вполне понятно - по названию и по тексту

7: В настройках чего - то есть эта параметр метода который нам передают , но не настройки
9: "персональные в этом подсчете" - пропущено слово сертификаты
13: вот здесь исходит от отрицания (тройное), а я бы построила наоборот, например: чтобы перс сертификаты применились (а он может быть применен не только один, а несколько сразу - что очень важно) должны быть соблюдены условия: выигрыш сертификата в арбитраже в лучшей комбинации, в том числе и перед другими сертификатами ( и дать ссылку на арбитраж) + телефон юзера и

я бы вообще ушла от отрицания в текстах (по стилистике текста), если это не прямой запрет - не должно быть...

  1. ты о чем - в меню интерфейсе нет такой настройки. В параметре метода есть, если она активирована, то сертификат можно использовать только предъявив номер телефона, к которому привязаны именные сертификаты

Саш, начала править, а потом поняла ,мне кажется - здесь логический принцип не тот
перс сертификат можно применить насколько я помню - без какого либо разрешения
если введен код сертификатов И и введен телефон юзера, владельца сертификатов, тогда сертификаты играют в арбитраже

Параметр в запросе " персональный сертификат" - это только для того, чтобы персональные сертификаты юзера ( код которого передан в запросе) участвовали в оплате и арбитраже без предъявления кодов
достаточно номера телефона и TRUE по параметру "use_personal_certificates"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

И мне кажется, если позволишь, можно проще - сейчас сложно читается
если ты говоришь про идентификатор, потом номер телефона - это как-то длинный путь

в запросах есть параметр "identifier"
в нем должен быть номер телефона юзера, на которого оформлен перс сертификат - только тогда он (они) становятся доступными к применению в заказе
или должен быть флаг, тогда\а номера сертификатов не надо пеедавать в запросе: система пойдет в юзера, увидит его сертификаты все и разрешит их применить в рамках возможностей по заказу

если что можем созвонится - может голосом проще?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants