Skip to content

Feat one yandex login #972

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

Merged
merged 30 commits into from
Apr 4, 2025
Merged

Feat one yandex login #972

merged 30 commits into from
Apr 4, 2025

Conversation

cin-bun
Copy link
Collaborator

@cin-bun cin-bun commented Apr 1, 2025

Checklist:

https://st.yandex-team.ru/DEVSHAD-252 base task
https://st.yandex-team.ru/DEVSHAD-54 contest issue

НУЖНО ДЕЛАТЬ МИГРАЦИЮ

  • The ticket number is indicated in the comments
  • Unrelated code has been removed (unused imports, print statements, unrelated template context variables, commented code, etc)
  • I translated the new messages and added only the .po files that actually changed.
  • If there were changes in the frontend, there is a PR in the front repository.

  • I have added tests that prove my code works
  • No common performance issues (e.g. N + 1 problem)

  • ЛК: теперь в карточке только одно поле Яндекс логин - которое подтверждается
  • ЛК: если его нет то так же отображается ссылка на привязку для юзера и надпись что его нет для кураторов
  • ЛК: если пользователь хочет поменять Яндекс логин он это может сделать через редактирование и надпись-ссылку [Изменить]
  • ЛК: куратор может удалить через админку обьект подтвержденного яндекс логин
  • ЛК: в админ форме редактирования поле Яндекс логин осталось, но значение берется из подтвержденного и поле не редактируемое + внизу есть полный список полей Яндекс логин
  • создание студента из абитура
    1. у нас два метода для этого протестировала начиная от заполнения формы согласия я только один, где второй на форме не знаю
    2. если у абитура уже есть пользователь и у него уже есть привязанный подтвержденный логин, то мы его оставляем старым, если что он сможет его поменять в лк
    3. если у нас не хватило каких либо данных для создания связки с подтверженным логином, то мы его просто не создаем и в аккаунте он остается пустым
  • контест: в выгрузках контеста используем подтверженный яндекс логин, если у пользователя он не заполнен то просто ничего не прорастет из контеста (на базовом протестировано, но по хорошему нужно потестить еще будет и последить за этим)
admin-edit edit_profile profile1 profile2 profile3 contest_score

@cin-bun cin-bun requested a review from Dmi4er4 April 2, 2025 07:17
Copy link
Collaborator

@Dmi4er4 Dmi4er4 left a comment

Choose a reason for hiding this comment

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

  1. Может ли человек с неподключенным логином добавить его через свою форму редактирования? Или он изменяет там, а подключает на основной странице с инфой?

  2. у нас два метода для этого протестировала начиная от заполнения формы согласия я только один, где второй на форме не знаю

Давай тут немного подробнее: какие есть 2 метода и какие сложности с нахождением одного из них?

  1. (на базовом протестировано, но по хорошему нужно потестить еще будет и последить за этим)

Давай тоже немного подробнее) Немного непонятно, что именно осталось сделать и почему это хотим делать после выкатки в прод)

@cin-bun
Copy link
Collaborator Author

cin-bun commented Apr 2, 2025

@Dmi4er4

  1. Чел может поменять свой логин только через кнопочку Изменить. Нажимая на нее он попадает в стандартный наш флоу смены логина через авторизацию Яндекс. Сама надпись логина в форме не кликабельна для изменения

  2. Метод create_student и create_student_from_applicant
    Вот для второго я нашла втюшку и ручку, но дёрнуть сходу не получилось и найти где эта кнопка тоже

  3. Я проверила только тот флоу до которого я догадалась дошла из кода и смогла сделать у себя на локалхосте. Это флоу отправки письма с ссылкой на форму подтверждения->ее заполнения -> создание пользователя и тд
    Так как инструкции нет никакой как этим пользоваться и какие сценарии как должны поддерживать и как это имплементировать у себя, я предполагаю, что я протестировала только базовый сценарий, и мб есть какие то ещё, которые из кода мне были не видны (сейчас цель сделать ревью и после обстучать все изменения от Катю с ее уточнениями и ОК )

@cin-bun cin-bun requested a review from Dmi4er4 April 3, 2025 19:50
@cin-bun cin-bun merged commit 59f62f6 into cscenter:master Apr 4, 2025
2 checks passed
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.

2 participants