[Hexlet#276]. Fix cors error. Update js script, update tests, add override handleMethodArgumentNotValid method #279
Conversation
# Conflicts: # src/main/resources/db/changelog/db.changelog-master.xml
…odArgumentNotValid method
…odArgumentNotValid method
|
Перед мерджем нужно будет поменять url для отправки запроса на api, ща урл с рендера стоит 'https://hexlet-correction-u17z.onrender.com/api/workspaces' |
|
|
|
Привет! Тест посмотрю, почему падает |
|
@Malcom1986 там тест падает из-за баги в одном из предыдущих пулл реквестов |
|
@kitdim привет! По сути к нам возвращается JSON с указанием где ошибка и в чём она заключается, надо пройти циклом и в зависимости от ключа, вписать в нужный div для ошибки сам текст предупреждения. |
|
@bazilval салют) |
|
@kitdim используй тогда для этих div-ов класс бутстрапа "invalid-feedback", чтобы было однообразно с нашим основным сервисом и аккуратно смотрелось |
src/widget/index.js
Outdated
| @@ -275,7 +299,7 @@ const handleTypoReporter = (options) => { | |||
| const initialState = { | |||
| modalShown: false, | |||
| options: { | |||
| workSpaceUrl: 'https://hexlet-correction.herokuapp.com/api/workspaces', | |||
| workSpaceUrl: 'https://hexlet-correction-u17z.onrender.com/api/workspaces', | |||
There was a problem hiding this comment.
Тут вы ссылку поменяли, давайте вернем назад :)
There was a problem hiding this comment.
Я так понял это для нас, чтобы демо работало :)
|
Да, насчет алерта согласен. Лучше на форме это выводить. |
| private final List<String> errors; | ||
|
|
||
| public ApiError(HttpStatus status, String message, List<String> errors) { | ||
| super(); |
There was a problem hiding this comment.
А у кого мы в этом случае вызываем super(), если класс ни от кого не наследуется? У Object? Если так, то нет нужды в явном вызове super()
src/widget/index.js
Outdated
| @@ -267,6 +278,19 @@ const isSelectionLeftToRight = (selection) => { | |||
| return range.collapsed; | |||
| } | |||
|
|
|||
| const isSuccessPost = (response) => { | |||
| return response.status === 201; | |||
There was a problem hiding this comment.
Возможно проверку в один оператор, которая используется один раз, можно оставить в изначальном виде и не выносить в отдельную функцию
@fey, на самом деле усложняет всё. потому-что бутстрап может быть не на каждом сайте. то что у нас в примере, идеальный случай. мб, лучше так отрисовывать? |
|
Я всё подготовил (демо теперь из-за смены урла не работает, можно выгрузить ветку и так посмотреть), но давайте на будующее этот момент тоже поправим. |
…odArgumentNotValid method
@kitdim насчёт того, что у нас бутстрап может быть не на каждом сайте это верная мысль! |
…odArgumentNotValid method
…odArgumentNotValid method
|
Сделал, можем сливать |
|
|
||
| targetElement.insertAdjacentElement('afterend', divElement); | ||
|
|
||
| const style = document.createElement('style'); |
There was a problem hiding this comment.
И всё-таки я бы прям добавил этот стиль в уже существующее в самом начале файла описание стилей
Чтобы просто всё оформление было в одном месте, для удобства поддержания
There was a problem hiding this comment.
Я сделал так, потому что мы не можем гарантировать что конечный пользователь ввёдет что-то неправильно (длинный эмаёл или описание). А так если хочешь, то можешь сам подправить.
Не прошёл один интеграционный тест. Тест недавно добавили. В предыдущем мердже он тоже не прошёл.
Демо