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

[ПЕРЕНОС] Добавлен страйкбольный лектер #350

Merged
merged 10 commits into from
Aug 27, 2024
Merged

[ПЕРЕНОС] Добавлен страйкбольный лектер #350

merged 10 commits into from
Aug 27, 2024

Conversation

Darkiich
Copy link
Contributor

@Darkiich Darkiich commented Aug 25, 2024 •
edited by Schrodinger71
Loading

Описание PR

Этот ПР переносить страйкбольный лектер
Медиа

image

Проверки

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

Изменения

🆑

  • add: Добавлен страйкбольный лектер

@github-actions github-actions bot added No C# Changes: Localization Изменение локализации labels Aug 25, 2024
Bolper
Bolper reviewed Aug 25, 2024
View reviewed changes
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.

Понатыкал то что вижу с первого раза. Не воспринимать в серьёз. @1Stepka1 жду тебя.

Resources/Locale/ru-RU/ADT/Objects/Weapons/Guns/guns.ftl Outdated Show resolved Hide resolved
Resources/Prototypes/ADT/tags.yml Outdated Show resolved Hide resolved
@Bolper
Copy link
Contributor

Bolper commented Aug 25, 2024

Потом сделаю нормальное ревью если Стёпка не опередит

@Bolper Bolper self-requested a review August 25, 2024 17:28
@github-actions github-actions bot added the Changes: Sprites Изменение спрайтов label Aug 25, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 25, 2024 •
edited
Loading

RSI Diff Bot; head commit 8e3e2ed merging into 9fe32bf
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/ADT/Objects/Weapons/Guns/Ammunition/Casings/airsoft_bibis.rsi

State Old New Status
base-spent Added
base Added

Edit: diff updated after 8e3e2ed

@Darkiich Darkiich requested a review from 1Stepka1 August 26, 2024 10:52
@Bolper
Copy link
Contributor

Bolper commented Aug 26, 2024

так Ревью нужно?
тогда сделаю

@Bolper
Copy link
Contributor

Bolper commented Aug 26, 2024

так Ревью нужно? тогда сделаю

Где проверки @Darkiich
Я не вижу галочки напротив ПР завершен. Зачем мне тогда делать ревью?

Bolper
Bolper reviewed Aug 26, 2024
View reviewed changes
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.

Ну а все же. Зачем целый отдельный файл strikeball. По базовым почему б не запихать

Resources/Prototypes/ADT/Entities/Objects/Weapons/Guns/Projectiles/projectiles.yml Outdated Show resolved Hide resolved
Resources/Textures/ADT/Objects/Weapons/Guns/Ammunition/airsoft_bibis.rsi/meta.json Outdated Show resolved Hide resolved
Resources/Prototypes/ADT/Entities/Objects/Weapons/Guns/Projectiles/projectiles.yml Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/ADT/Objects/Weapons/Guns/guns.ftl Outdated Show resolved Hide resolved
Resources/Prototypes/ADT/Entities/Objects/Weapons/Guns/Rifels/rifles.yml Show resolved Hide resolved
Resources/Locale/ru-RU/ADT/Objects/Weapons/Guns/guns.ftl Outdated Show resolved Hide resolved
@Bolper
Copy link
Contributor

Bolper commented Aug 27, 2024 •
edited
Loading

Ну а все же. Зачем целый отдельный файл strikeball. По базовым почему б не запихать

Ну по этому поводу всё так же вопрос.

@Bolper
Copy link
Contributor

Bolper commented Aug 27, 2024

Ну а все же. Зачем целый отдельный файл strikeball. По базовым почему б не запихать

Ну по этому поводу всё так же вопрос.

Каждый раз создавать новый файл strike ball не оч хорошо наверно

@Darkiich
Copy link
Contributor Author

Ну а все же. Зачем целый отдельный файл strikeball. По базовым почему б не запихать

Ну по этому поводу всё так же вопрос.

Ну всмысле зачем, это же файл с прототипами магазинов

@Bolper
Copy link
Contributor

Bolper commented Aug 27, 2024

Ну а все же. Зачем целый отдельный файл strikeball. По базовым почему б не запихать

Ну по этому поводу всё так же вопрос.

Ну всмысле зачем, это же файл с прототипами магазинов

А не. Всё ок. Туплю

Bolper
Bolper previously approved these changes Aug 27, 2024
View reviewed changes
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.

Ну особо бед здесь не вижу

@Darkiich
Copy link
Contributor Author

Ну а все же. Зачем целый отдельный файл strikeball. По базовым почему б не запихать

Ну по этому поводу всё так же вопрос.

Ну всмысле зачем, это же файл с прототипами магазинов

А не. Всё ок. Туплю

Млядец

@Darkiich
Copy link
Contributor Author

Ну особо бед здесь не вижу

Люблю, целую

@Bolper Bolper dismissed their stale review August 27, 2024 10:43

Так стоп

Bolper
Bolper previously approved these changes Aug 27, 2024
View reviewed changes
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.

Так. Норм всё же

@Darkiich
Copy link
Contributor Author

Так. Норм всё же

XD

@Bolper
Copy link
Contributor

Bolper commented Aug 27, 2024

Так. Норм всё же

XD

Да падажди ты

@Bolper Bolper dismissed their stale review August 27, 2024 10:55

Че то мне не нравится. Чуть позже.

@Darkiich
Copy link
Contributor Author

Так. Норм всё же

XD

Да падажди ты

Огузок

Bolper
Bolper requested changes Aug 27, 2024
View reviewed changes
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.

Не.. Я тут себе голову сломал. Еще папки плодить не хочу. Ну это пиздец

Resources/Locale/ru-RU/ADT/Objects/Weapons/Guns/strikeball.ftl Outdated Show resolved Hide resolved
...cale/ru-RU/ADT/prototypes/Entities/Objects/Weapons/Guns/Ammunition/Cartridges/strikeball.ftl Outdated Show resolved Hide resolved
...ocale/ru-RU/ADT/prototypes/Entities/Objects/Weapons/Guns/Ammunition/Magazines/strikeball.ftl Outdated Show resolved Hide resolved
Resources/Prototypes/ADT/Entities/Objects/Weapons/Guns/Projectiles/projectiles.yml Outdated Show resolved Hide resolved
...ale/ru-RU/ADT/prototypes/Entities/Objects/Weapons/Guns/Ammunition/Projectiles/strikeball.ftl Outdated Show resolved Hide resolved
Resources/Textures/ADT/Objects/Weapons/Guns/Ammunition/airsoft_bibis.rsi/meta.json Outdated Show resolved Hide resolved
@Bolper
Copy link
Contributor

Bolper commented Aug 27, 2024

Так. Норм всё же

XD

Да падажди ты

Огузок

Да. Огузище

@Darkiich
Copy link
Contributor Author

@Bolper Ну получается ревью сделано

Bolper
Bolper reviewed Aug 27, 2024
View reviewed changes
Resources/Prototypes/ADT/Entities/Objects/Weapons/Guns/Ammunition/Projectiles/BulletBibis.yml Outdated
@@ -0,0 +1,35 @@
- type: entity
id: ADTBaseBulletRubber
Copy link
Contributor

Choose a reason for hiding this comment

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

Не не не
Я посмотрел. Базы находятся в папке выше, тобишь там где было раньше.

Copy link
Contributor

Choose a reason for hiding this comment

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

Resources/Prototypes/Entities/Objects/Weapons/Guns/Projectiles/projectiles.yml

Copy link
Contributor

Choose a reason for hiding this comment

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

Это у них так сделано. В нашем случае это в папке adt

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Не не не Я посмотрел. Базы находятся в папке выше, тобишь там где было раньше.

Огузок

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Не не не Я посмотрел. Базы находятся в папке выше, тобишь там где было раньше.

Огузок

Но я тебя прощаю :3

Copy link
Contributor

Choose a reason for hiding this comment

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

У эээ. Так то я тебе в прошлом ревью указал на айдишник который надо перенести было. Это твоя была инициатива и это переносить.
Я тебя тоже люблю~

@Darkiich Darkiich requested a review from Bolper August 27, 2024 13:56
@Darkiich
Copy link
Contributor Author

@Bolper

Bolper
Bolper approved these changes Aug 27, 2024
View reviewed changes
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.

Люблю обожаю

@Darkiich
Copy link
Contributor Author

Люблю обожаю

Люблю, целую

@Darkiich Darkiich added this pull request to the merge queue Aug 27, 2024
Merged via the queue into AdventureTimeSS14:master with commit 1343766 Aug 27, 2024
16 checks passed
@Darkiich Darkiich deleted the strikball branch August 27, 2024 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bolper Bolper Bolper approved these changes

@1Stepka1 1Stepka1 Awaiting requested review from 1Stepka1

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

Successfully merging this pull request may close these issues.
3 participants
@Darkiich @Bolper @1Stepka1