Hexlet / hexlet-correction

Typo Reporter
https://fixit.hexlet.io/
GNU Affero General Public License v3.0
41 stars 72 forks source link

fix bug #203 #232

Closed sushilyaz closed 6 months ago

sushilyaz commented 6 months ago

fix bug #203 replace dropdown list to input

fey commented 6 months ago

Задеплой те плз демку, если будут вопросы - спрашивайте. На них ответит @Malcom1986 =)

sushilyaz commented 6 months ago

Задеплой те плз демку, если будут вопросы - спрашивайте. На них ответит @Malcom1986 =)

Это мой первый PR, ты имеешь в виду на рендер залить? Если рендер, то вот: https://hexlet-cor.onrender.com

sushilyaz commented 6 months ago

Задеплой те плз демку, если будут вопросы - спрашивайте. На них ответит @Malcom1986 =)

Это мой первый PR, ты имеешь в виду на рендер залить? Если рендер, то вот: https://hexlet-cor.onrender.com

fey commented 6 months ago

Посмотрел, не хватает валидации емейла, тк можно вписать что угодно.

Malcom1986 commented 6 months ago

@sushilyaz Вот тут осталось от прежней страницы: Choose a user from the list

Снимок экрана 2023-12-27 в 12 57 49
sushilyaz commented 6 months ago

@sushilyaz Вот тут осталось от прежней страницы: Choose a user from the list Снимок экрана 2023-12-27 в 12 57 49

Ок, сегодня нормально поправлю страницу и сделаю валидацию

sushilyaz commented 6 months ago

@sushilyaz Вот тут осталось от прежней страницы: Choose a user from the list Снимок экрана 2023-12-27 в 12 57 49

Поправил страницу и добавил валидацию на email (хотя если честно я не понимаю, зачем это, если там и так будет поиск пользователя по email), но раз попросили, я добавил, демка выше @Malcom1986

fey commented 6 months ago

Поправил страницу и добавил валидацию на email (хотя если честно я не понимаю, зачем это, если там и так будет поиск пользователя по email)

Пользователь может ошибиться и ввести немеейл. Вообще все равно полезно добавлять валидацию на подобные поля.

sushilyaz commented 6 months ago

@Malcom1986 Еще актуально? Не смотрели?

Malcom1986 commented 6 months ago

@sushilyaz Привет! Тут выходные были, сегодня-завтра посмотрю