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

Проблемы при изменении данных пользователя в профиле #200

Closed
kairina2020 opened this issue Aug 30, 2023 · 7 comments

Comments

@kairina2020
Copy link

@fey , Тестировала профиль пользователя на внесение в него изменений.
Опишу в общем проблемы, которые встретились. Они повторяют те, что были при регистрации, поэтому сразу обратите внимание на них при внесении изменений.

  1. На мой взгляд подтверждение Email тут лишнее, так как при изменении любого из параметров, будь-то Username, фамилия имя или Email, каждый раз приходится вводить подтверждение адреса электронной почты. Велика вероятность ошибиться и неудобно.
  2. E-mail и его подтверждения остались регистрозависимыми, достаточно одной букве быть в другом регистре, сразу несовпадение.
  3. Позволяет успешно изменять верный Email на неверный/неполный по шаблону %@% без точки и кода страны.
  4. Заметила одну проблему, редко проявляется, возможно не критично, но для информации.
    При изменении Username, если измененный уже зарегистрирован в системе, то появляется сообщение об этом,
    НО иногда все же изменить можно на уже зарегистрированный.
    Проявляется редко, логику не получилось понять.
    Думала зависит от того, проходила ли полная регистрация с этим Username или только редактировался в профиле пользователя, но нет, НЕ зависит.
@TonyMudRec
Copy link
Contributor

могу взять

@fey
Copy link
Collaborator

fey commented Oct 16, 2023

@TonyMudRec здесь описаны несколько задач. Я предлагаю идти поэтапно. Давайте сперва поправим валидацию.

@fey
Copy link
Collaborator

fey commented Oct 16, 2023

@kairina2020 можете описать поэтапно, как вы обновляли профиль? Я сейчас попробвоал и получаю ошибку
image

@TonyMudRec
Copy link
Contributor

пока не буду брать

@fey
Copy link
Collaborator

fey commented Oct 19, 2023

Пардон, я не тот проект посмотрел 🤦

@fey
Copy link
Collaborator

fey commented Oct 19, 2023

На мой взгляд подтверждение Email тут лишнее, так как при изменении любого из параметров, будь-то Username, фамилия имя или Email, каждый раз приходится вводить подтверждение адреса электронной почты. Велика вероятность ошибиться и неудобно.

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

Позволяет успешно изменять верный Email на неверный/неполный по шаблону %@% без точки и кода страны.```

ага, валидация должна быть везде одинаковой.

@fey
Copy link
Collaborator

fey commented Oct 19, 2023

НО иногда все же изменить можно на уже зарегистрированный.
Проявляется редко, логику не получилось понять.
Думала зависит от того, проходила ли полная регистрация с этим Username или только редактировался в профиле пользователя, но нет, НЕ зависит.```

по идее юзернейм может быть неуникальным, хм. Мы пока нигде не выводим юзернеймы, но надо это будет чекнуть.

@fey fey closed this as completed Mar 11, 2024
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

No branches or pull requests

3 participants