Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Время на младшие должности #333

Merged
merged 2 commits into from
Aug 23, 2024

Conversation

jungarikjan
Copy link
Contributor

@jungarikjan jungarikjan commented Aug 22, 2024

Описание PR

  • Убрано ограничение на становление кадетом, тех. ассистентом, интерном, лаборантом, если ты уже много наиграл в отделе
  • Для того, чтобы стать грузчиком или тех. ассистентом - нужно набрать 4 часа (анти-набегаторские меры)

Медиа

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑

  • remove: Убрано ограничение на становление кадетом, тех. ассистентом, интерном, лаборантом, если ты уже много наиграл в отделе
  • tweak: Для того, чтобы стать грузчиком или тех. ассистентом - нужно набрать 4 часа

Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше закомменть, то что удалил и поставь ADT Tweak в комментарии

@github-actions github-actions bot added Status: Awaiting Changes Ожидание изменений and removed Status: Needs Review labels Aug 22, 2024
@Bolper
Copy link
Contributor

Bolper commented Aug 22, 2024

Лучше закомменть, то что удалил и поставь ADT Tweak в комментарии

Душить плохо, но я плохой и душный
Но аппрув после этого дам

@jungarikjan
Copy link
Contributor Author

Поменял если что

@github-actions github-actions bot removed the Status: Awaiting Changes Ожидание изменений label Aug 22, 2024
@Darkiich
Copy link
Contributor

Лучше закомменть, то что удалил и поставь ADT Tweak в комментарии

Душить плохо, но тут ты правильно написал, что надо сделать

Copy link
Contributor

@Darkiich Darkiich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Джунгар, в описании ПРа у значка CL желательно написать ник, чтобы оно отображалось в ченджлогах(может оно итак будет отображать, но на всякий думаю стоит)

@github-actions github-actions bot added the Status: Awaiting Changes Ожидание изменений label Aug 22, 2024
@Darkiich Darkiich dismissed their stale review August 23, 2024 00:00

Эмэмэ, немного перепутал кнопки

Copy link
Contributor

@Bolper Bolper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я конечно утром и нихрена не вижу. Но вроде всё ок и Стёпка дал апрув.

@github-actions github-actions bot removed the Status: Awaiting Changes Ожидание изменений label Aug 23, 2024
@PyotrIgn PyotrIgn added this pull request to the merge queue Aug 23, 2024
Merged via the queue into master with commit ab5a362 Aug 23, 2024
17 checks passed
@PyotrIgn PyotrIgn deleted the assistant-and-salvage-time branch August 23, 2024 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants