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.

Sign up for GitHub

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

Jump to bottom

Minor changes to the names and descriptions of roles and departments #87

Merged
merged 4 commits into from
Sep 7, 2024
Merged

Minor changes to the names and descriptions of roles and departments #87

merged 4 commits into from
Sep 7, 2024

Conversation

SpyDev14
Copy link

@SpyDev14 SpyDev14 commented Sep 6, 2024

Description

  • Исправил регистры букв ролей, теперь всё что должно быть в нижнем регистре - в нижнем регистре.
  • Переименовал несколько ролей, например "племенной целитель" заменил на "племенной знахарь" и т.п.
  • Заменил некоторые описания ролей, увидить всё можете во вкладке изменений. В кратце - улучшил описания.
  • Поменял некоторые значения job-supervisiors. Например теперь "обязанность" выживших (мусорщики и т.п) - не умереть от радтаракана.

Багов нет, всё работает.

  • Completed Task

Мой первый PR 🥰

@SpyDev14
06.09.24 22:34 GMP+4
73c3ee7 
Rename & fix all job names & a little change job-supervisiors description. 06.09.24 22:34 GMP+4
@SpyDev14
07.09.24 1:51 GMT+4
6cf88db 
same thing
@SpyDev14
07.09.24 2:17 GMT+4
adf327e
@SpyDev14
rename ?znahar to tribal ?znahar
f10015d
@SpyDev14
Copy link
Author

SpyDev14 commented Sep 6, 2024

Буээ, какие-то автотесты не прошли. Хмм, значит что-то не так.

@SpyDev14
Copy link
Author

SpyDev14 commented Sep 6, 2024

Или так и надо, я не знаю.

Zekins3366
Zekins3366 approved these changes Sep 7, 2024
View reviewed changes
Copy link
Collaborator

@Zekins3366 Zekins3366 left a comment

Choose a reason for hiding this comment

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

Нормальный перевод

@Zekins3366
Copy link
Collaborator

@VlaDOS1408 Обнови до актуальной версии сборку. Указываю на первый чек ошибку Check Merge Conflicts

@Zekins3366
Copy link
Collaborator

Ну вот теперь нормально

@Zekins3366 Zekins3366 merged commit 60e1aee into Corvax-Frontier:master Sep 7, 2024
6 checks passed
@SpyDev14
Copy link
Author

SpyDev14 commented Sep 7, 2024 •
edited
Loading

Ура, победа

@SpyDev14 SpyDev14 deleted the minor-changes-to-the-names-and-descriptions-of-roles-and-departments branch September 7, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Zekins3366 Zekins3366 Zekins3366 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SpyDev14 @Zekins3366