[TL-4694] add buttons for sending vacancy to moderation and to archive #753
Conversation
Просто шла мимо, но есть вопрос: зачем тебе здесь в названии вип? ПР в драфте сам по себе намекает, что предлагаемые изменения могут быть не совсем верны или не до конца доработаны |
|
согласен с Яной, притом тут WIP в имени коммита, если сейчас принять - то это улетит в main ветку |
Senyarsenie
force-pushed
the
send-vacancy-to-moderation
branch
from
8db28f9
to
ba52a12
Compare
| @@ -0,0 +1,6 @@ | |||
| .fs-5 = t('.additional_actions') | |||
There was a problem hiding this comment.
Предлагаю если вакансию нельзя за архивировать или отправить на модерацию не показывать эти кнопки
|
|
||
| private | ||
| def archive | ||
| vacancy = current_user.vacancies.find params[:id] |
There was a problem hiding this comment.
Нужен редирект так как сейчас когда отправляешь вакансию в архив страница не обновляется и кнопка архивации остается активна и при повторном нажатии страница падает
| def change_visibility(vacancy) | ||
| vacancy.archive! if params[:archive] | ||
| def send_to_moderate | ||
| vacancy = current_user.vacancies.find params[:id] |
There was a problem hiding this comment.
и было бы не плохо флешь сообщение пользователю об успешной архивации
|
Еще есть несколько вопросов по бизнес логике: 1 Будет ли возможность у пользователя достать вакансию из архива, сейчас для этого помоему возможности нет, хотя event в стейт машине есть |
|
|
@usernaimandrey давай не давать редактировать архивные вакансии |
Нужно ди тогда архивные вакансии вот тут пользователю отображать? |
|
А сейчас как, пользователь видит свои вакансии в архиве у себя в интерфейсе? |
Да видит |
|
тогда пускай остается |
|
@Senyarsenie привет подскажи как тут у тебя дела? Есть ли какие то трудности? |
No description provided.