[#248] Fix login validation bug in modals #250
Merged
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.
[#248 ]
Переезд с [#249], пришлось переименовать ветку, чтобы задеплоиться на render.
Ссылка: https://supersnowsnail-runit.onrender.com/
Привёл валидацию имени пользователя в модалке редактирования профиля к требуемому виду, аналогично #243.
Заодно пофиксил тот же самый баг в модалке регистрации (которая выпадает, если попробовать поделиться сниппетом, будучи незалогинненым, см. скрин).
Бага там было сразу два: собственно, сама валидация, а также то, что у поля для логина был name="name" вместо name="login", из-за чего из формы отсылались неверные данные и сервер возвращал 400 ошибку. По итогу поле логина переделал по аналогии с тем же полем в frontend/src/Pages/SignUp.jsx (кстати, ещё опечатку поправил в catch валидации модалки, там 409 ошибку ловило, поставил 400 по аналогии с frontend/src/Pages/SignUp.jsx).
Ещё, помимо прочего, обнаружил ошибку в валидации на стороне бекэнда (в backend/src/users/dto/create-user.dto.ts и backend/src/users/dto/update-user.dto.ts), причём ранее там были прописаны вообше не валидные RegEx, из-за чего это всё равно ни на что не влияло. На всякий случай исправил RegEx и критерии длины логина на новые, хотя при штатном использовании приложения валидация на фронте всё равно не пропустила бы невалидные данные.