hexlet-correction icon indicating copy to clipboard operation
hexlet-correction copied to clipboard

Published 20 hours ago • verified publisher icon Hexlet

Rename the Open button

Open ssssank opened this issue 1 year ago • 12 comments

Problem description

It is unclear what the Open button does in the sent typo

Proposed solution

Since this button translates the typo to the status "In progress", you can rename it to "Start work" or "Get started"

ssssank avatar Jul 11 '23 12:07 ssssank

Актуально

fey avatar Mar 11 '24 13:03 fey

Привет! Я заканчиваю курс и хочу попробовать смогу ли разобраться в коде.

nuuska-muikkunen avatar Mar 15 '24 06:03 nuuska-muikkunen

Да, берите.

fey avatar Mar 15 '24 06:03 fey

Имя кнопки OPEN определяется именем константы OPEN перечисления TypoEvent. В зависимости от этих констант (OPEN, RESOLVE, REOPEN, CANCEL) меняется статус переданной опечатки, задается имя кнопки и ее цвет.

  1. Будем переименовывать OPEN в START WORK (я бы лучше переименовал просто в START)?
  2. Надо ли, следуя нашей логике, переименовать константу REOPEN в RESTART (RESTART WORK)?
  3. Имена методов тестирования тоже переименовывать (patchTypoEventOpenToReported и patchTypoEventReopenToCanceled)?
  4. Русский вариант тоже меняю - НАЧАТЬ (НАЧАТЬ РАБОТУ), ПЕРЕДЕЛАТЬ - перевод для RESTART?

PS Спрашиваю, поскольку у меня это первый опыт работы с незнакомым кодом.

nuuska-muikkunen avatar Mar 20 '24 18:03 nuuska-muikkunen

@Malcom1986 поможешь нам? Скорее всего по коду тоже нужно поменять, ведь у нас и смыслы меняются.

fey avatar Mar 21 '24 08:03 fey

@nuuska-muikkunen Приветствую. Давайте сами состояния трогать не будем, оставим как есть. Они довольно семантично называются. Поменяем только надпись на кнопке, сделаем ее более юзер френдли

Malcom1986 avatar Mar 21 '24 11:03 Malcom1986

Состояния не трогаю. Только названия ивентов, от которых названия кнопок определяются. Вместо OPEN - START, вместо REOPEN - RESTART. На русском: вместо ОТКРЫТЬ - НАЧАТЬ, вместо ОТКРЫТЬ ЗАНОВО - НАЧАТЬ СНОВА.

nuuska-muikkunen avatar Mar 21 '24 14:03 nuuska-muikkunen

Вот так выглядят новые кнопки (см прилагаемые скринки экранов с локального запуска приложения). result of screens issue #177 button OPEN rename.pdf Можно коммитить? Изменения в отдельной ветке.

nuuska-muikkunen avatar Mar 21 '24 17:03 nuuska-muikkunen

Вы можете запушить в свою ветку и создать пулл реквест https://www.youtube.com/watch?v=pUT3mx1ZRUM Пока ПР не принят, в него можно спокойно вносить изменения. При принятии ПРа будут приняты изменения на момент принятия. Скрин не могу посмотреть, он в формате ПДФ. Лучше задеплойте демо и скиньте ссылку на посмотреть.

fey avatar Mar 21 '24 18:03 fey

Деплой не проходит. У вас Dockerfile не менялся 9 месяцев и все про Maven, а POM file убрали при миграции на gradle. Я правильно понял?

nuuska-muikkunen avatar Mar 22 '24 07:03 nuuska-muikkunen

Да, здорово будет, если и докерфайл поправите заодно. Можно в Procfile подглядывать, как мы заускаем приложение на хероку

Malcom1986 avatar Mar 22 '24 07:03 Malcom1986

Наконец победил деплой :-) Можете смотреть результат https://hexlet-correction-rqc5.onrender.com/. У меня требуется VPN для работы.

nuuska-muikkunen avatar Mar 24 '24 11:03 nuuska-muikkunen