search

Hexlet / hexlet-correction

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

Update signup with empty names #299

Closed uchitsa closed 3 months ago

uchitsa commented 3 months ago

@fey @dzencot hi, please check PR

SibirBear commented 3 months ago

@uchitsa, привет! Проверь свое решение, аннотация @Size подразумевает, что размер значения должно быть от 1 и больше, другими словами необходимо все равно указать 1 знак (хоть пробел) чтобы успешно зарегистрироваться.

Кроме того, такой импорт "import lombok.*;" не пройдет проверку линтера, необходимо использовать построчный импорт.

Тест проверяет количество записей в бд - это условие не гарантирует того, что аккаунт был создан. Возможна ситуация, когда в бд уже есть запись, например админа, и тогда тест пройдет или наоборот - вбд не пустая. Лучше проверить был ли создан аккаунт в бд, например по email - обязательному параметру.

uchitsa commented 3 months ago

@SibirBear спасибо! Обновлено

fey commented 3 months ago

@uchitsa hi, deploy please demo to render

SibirBear commented 3 months ago

Проблема в том, что при создании аккаунта используется модель Account у которой аналогичные ограничения, что и у SignupAccountModel. Нужно изменить аннотации у модели Account и поправить тест - в нем ошибка в последней строчке. Рекомендую беред комитами запускать проверку линтера и тесты.

uchitsa commented 3 months ago

@fey what does it mean "deploy please demo to render"? please describe

uchitsa commented 3 months ago

@Malcom1986 привет, просьба проверить ПР

fey commented 3 months ago

@uchitsa задеплойте приложение на render.com, чтобы можно было посмотреть изменения в вебе, потыкать кнопки, проверить работу форм и приложения.

uchitsa commented 3 months ago

@fey просьба скинуть мануал, инфо, ссылку, как задеплоить проект на render.com корректно

uchitsa commented 3 months ago

@fey 1 попытка прошла с ошибкой Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'liquibase' defined in class path resource [org/springframework/boot/autoconfigure/liquibase/LiquibaseAutoConfiguration$LiquibaseConfiguration.class]: java.lang.RuntimeException: Driver org.postgresql.Driver claims to not accept jdbcUrl, ${JDBC_DATABASE_URL}

fey commented 3 months ago

@uchitsa у меня нет под рукой рабочего мануала. по идее тут обычное spring приложение. можете написать ребятам, которые делоили, например тут https://github.com/Hexlet/hexlet-correction/pull/284 @niyatanya

SibirBear commented 3 months ago

@uchitsa в настройках приложения на Render нужно указать переменные для доступа к базе данных, для этого нужно создать базу данных Postgres на Render и перенести необходимые значения в настройки своего приложения. Приложение подключить из своего репозитоия на github. Переменные должны быть: JDBC_DATABASE_URL JDBC_DATABASE_USERNAME JDBC_DATABASE_PASSWORD

Вот статья в помощь https://ru.hexlet.io/blog/posts/render-java

uchitsa commented 3 months ago

@SibirBear полезный навык +1, спасибо

uchitsa commented 3 months ago

https://hexlet-correction-dl6y.onrender.com it works! Please check PR @fey @Malcom1986 @SibirBear

Malcom1986 commented 3 months ago

Ага, все работает. Поправьте еще пожалуйста пару земечаний по коду и будем мержить

uchitsa commented 3 months ago

@Malcom1986 обновлено с учётом замечаний

fey commented 3 months ago

@uchitsa давайте еще форму обновим, удалим поля image