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

add mirotvorec, fixed some stuff #490

Merged

Conversation

lulpiopio
Copy link
Contributor

добавил миротворца (попросили меня)
добавил себя и кента в бебеплинк
добавил кенту еще один звук
дал кд на звуки 20 секунд

Кратное описание

Я предлагаю добавить МЭМЫ

По какой причине

Потому что я могу

Медиа(Видео/Скриншоты)

Эксклюзивно только.

Проверки

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

Changelog

🆑 lulpiopio

  • add: Добавил Миротворца.
  • add: Добавил еще один звук и это не Папич.

добавил миротворца (попросили меня)
добавил себя и кента в бебеплинк
добавил кенту еще один звук
дал кд на звуки 20 секунд
Copy link
Contributor

github-actions bot commented Oct 11, 2024

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

Resources/Textures/_Sunrise/Objects/Fun/toys.rsi

State Old New Status
plushie-peacemaker Added

Edit: diff updated after aa512c9

Copy link
Contributor

@Rinary1 Rinary1 left a comment

Choose a reason for hiding this comment

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

Хуйня миша, всё по новой

This comment was marked as resolved.

This comment was marked as resolved.

Comment on lines +693 to +694
description: Там где он, там мир, но там где война, там он несëт смерть.
Copy link
Contributor

Choose a reason for hiding this comment

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

локализация в .yml ай-ай-ай. А фиксить кто будет? Отныне русские символы в прототипах караються. Локализацию делать только в Locale/ru-Ru по соответсвующим папкам/файлам, в твоем случае это
Locale/ru-RU/_prototypes/_sunrise/objects/fun/toys.ftl.

- type: entity
parent: BasePlushie
id: PlushieMirotvorec

This comment was marked as resolved.


- type: listing
id: UplinkSunrisePlushieMirotvorec
productEntity: PlushieMirotvorec

This comment was marked as resolved.

stock: 1

- type: listing
id: UplinkSunrisePlushieMirotvorec

This comment was marked as resolved.

@lulpiopio
Copy link
Contributor Author

да миша ты ёбу дал, я лингвист, зач нужно это?

@Rinary1
Copy link
Contributor

Rinary1 commented Oct 11, 2024

да миша ты ёбу дал, я лингвист, зач нужно это?

Кроме тебя есть другие разработчики, + эта id будет в игре. Это как минимум уважение к другим. А не тяп ляп.

На счёт локализации. Sunrise хочет ввести свап языков.

@lulpiopio
Copy link
Contributor Author

я не разраб, это не мое дело локализацию вводить, пусть займется человек, который назначен на это и в последствии оформит как должно быть

атм я сделал так, как в коде было на примерах

я могу же ща как-то название сменить прямо здесь?

@Rinary1
Copy link
Contributor

Rinary1 commented Oct 11, 2024

я не разраб, это не мое дело локализацию вводить, пусть займется человек, который назначен на это и в последствии оформит как должно быть

атм я сделал так, как в коде было на примерах

я могу же ща как-то название сменить прямо здесь?

Если ты не разраб и не готов получать корректировки, не делай PR'ы пожалуйста, это только усложнит нам жизнь.

@Rinary1
Copy link
Contributor

Rinary1 commented Oct 11, 2024

Когда ты делаешь PR ты должен быть готовым что получить требования в правках. Тебе же сказали, что локализация в прототипах это не норма, а ты скидываешь на других. Нам не платят что-бы мы фиксили дерьмо за другими.

@lulpiopio
Copy link
Contributor Author

Когда ты делаешь PR ты должен быть готовым что получить требования в правках. Тебе же сказали, что локализация в прототипах это не норма, а ты скидываешь на других. Нам не платят что-бы мы фиксили дерьмо за другими.

уважаемый, прочитай мое сообщение и дай ответ: Я МОГУ ЭТО ЗДЕСЬ КАК-ТО ИЗМЕНИТЬ?

@AverageNotDoingAnythingEnjoyer
Copy link
Contributor

Когда ты делаешь PR ты должен быть готовым что получить требования в правках. Тебе же сказали, что локализация в прототипах это не норма, а ты скидываешь на других. Нам не платят что-бы мы фиксили дерьмо за другими.

уважаемый, прочитай мое сообщение и дай ответ: Я МОГУ ЭТО ЗДЕСЬ КАК-ТО ИЗМЕНИТЬ?

Здесь это где, на самой странице гитхаба?

@Rinary1
Copy link
Contributor

Rinary1 commented Oct 11, 2024

Я МОГУ ЭТО ЗДЕСЬ КАК-ТО ИЗМЕНИТЬ?

Если ты про гитхаб, да, просто выбераешь файл для редактирования и там будет кнопка в троиточии.

Так-же можешь просто запушить изменения в эту же ветку.

@lulpiopio
Copy link
Contributor Author

транслитерации нет более, everything in english with correct words

@VigersRay VigersRay merged commit 73d44d4 into space-sunrise:master Oct 11, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants