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

акшион связи для ИИ #426

Closed
wants to merge 5 commits into from
Closed

акшион связи для ИИ #426

wants to merge 5 commits into from

Conversation

Darkiich
Copy link
Contributor

@Darkiich Darkiich commented Sep 7, 2024 •
edited
Loading

Описание PR

Ссылка на публикацию в Discord

Медиа

Проверки

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

Изменения

image
image

🆑 Дубик

  • add: ИИ научился делать оповещения

@github-actions github-actions bot added No C# Changes: Localization Изменение локализации labels Sep 7, 2024
Schrodinger71
Schrodinger71 reviewed Sep 7, 2024
View reviewed changes
Copy link
Collaborator

@Schrodinger71 Schrodinger71 left a comment

Choose a reason for hiding this comment

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

На локалке проверял?

@Schrodinger71
Copy link
Collaborator

Schrodinger71 commented Sep 7, 2024 •
edited by Darkiich
Loading

Скрины бы

@Darkiich
Copy link
Contributor Author

Darkiich commented Sep 7, 2024

На локалке проверял?

Конечно

@Darkiich
Copy link
Contributor Author

Darkiich commented Sep 7, 2024

Скрины бы

Добавил

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 7, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Schrodinger71
Schrodinger71 reviewed Sep 7, 2024
View reviewed changes
Resources/Locale/en-US/job/job-names.ftl Outdated
@@ -33,7 +33,6 @@ job-name-botanist = Botanist
job-name-bartender = Bartender
job-name-passenger = Passenger
job-name-salvagespec = Salvage Specialist
job-name-station-ai = Station AI
Copy link
Collaborator

Choose a reason for hiding this comment

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

а локаль зачем удалять, лучше не трогать совсем ничего в этих файлах

Schrodinger71
Schrodinger71 requested changes Sep 7, 2024
View reviewed changes
Copy link
Collaborator

@Schrodinger71 Schrodinger71 left a comment

Choose a reason for hiding this comment

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

При отправке оповещения поле Отправителя пустое, туда или добавить что-то или убрать его

Локаль английскую можно не удалять, должно всё итак работать

@Schrodinger71
Copy link
Collaborator

так PR: от сюда space-wizards/space-station-14#31852

@Darkiich
Copy link
Contributor Author

Darkiich commented Sep 7, 2024

так PR: от сюда space-wizards/space-station-14#31852

Не, я это сам делал

@Schrodinger71 Schrodinger71 marked this pull request as draft September 7, 2024 14:54
@Bolper
Copy link
Contributor

Bolper commented Sep 7, 2024

Эт в драфте, пинганете когда ревью нужно будет

@Darkiich
Copy link
Contributor Author

Darkiich commented Sep 7, 2024

Опоздал короче, к визардам отправили это уже

@Darkiich Darkiich closed this Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Schrodinger71 Schrodinger71 Schrodinger71 requested changes

@1Stepka1 1Stepka1 Awaiting requested review from 1Stepka1

@Bolper Bolper Awaiting requested review from Bolper

Assignees
No one assigned
Labels
Changes: Localization Изменение локализации No C# Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Darkiich @Schrodinger71 @Bolper @ScarKy0