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

PipeSound #936

Closed
wants to merge 3 commits into from
Closed

PipeSound #936

wants to merge 3 commits into from

Conversation

Helm4142
Copy link

@Helm4142 Helm4142 commented Apr 23, 2024

Описание PR

Сделал по трекеру новый звук трубы, когда падает на тайл воспроизводится звук трубы.
https://ptb.discord.com/channels/1097181193939730453/1231526588911325216

Медиа

default.mp4

Проверки

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

Изменения

🆑 Helm

  • tweak: Изменён звук трубы при падении на тайл!

Сделал по трекеру новый звук трубы, когда падает на тайл воспроизводится звук трубы.
@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Apr 23, 2024
@Helm4142
Copy link
Author

Какие-то ошибки повылезали, увы...

@ZoNeSRuS
Copy link

Какие-то ошибки повылезали, увы...

Чини ошибки хули

@Helm4142
Copy link
Author

Какие-то ошибки повылезали, увы...

Чини ошибки хули

А как... Я не умею у меня лапки.

@Helm4142
Copy link
Author

Вообще всё работает отлично, а ошибки ошибаются. Без сторонней помощи я никак не смогу это пофиксить...

@Helm4142
Copy link
Author

У меня в компоненте каких-то какашек не хватает, если в игре найду каких то попытаюсь допилить, но скорее всего я ничё не смогу сделать.

Почитал, были какие-то ошибки со следующим компонентом, решил просто переместить хд
@Helm4142
Copy link
Author

ЮХУ!!! Быть таким глупым как я это ужас что-бы вставить 2 строчки готового кода у меня ушло 2 дня!

@DexlerXD
Copy link

@Ady4ik это одобрено? это же -уши

@Helm4142
Copy link
Author

@Ady4ik это одобрено? это же -уши

Насчёт -уши, могу сегодня через код уменьшить звук, через сайт пытался но там либо тихо либо громко, усреднённое выбрал

Пощажу ваши ушки, потыркал посмотрел и оглох
@Helm4142
Copy link
Author

Ломал голову час как изменить код что-бы сделать тише звук

@EstKemran
Copy link

@Helm4142 Ты не обновил медиа после убавления громкости. Сделать бы, что бы ревьюер смог оценить

Copy link

@Kit0vras Kit0vras left a comment

Choose a reason for hiding this comment

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

sonic-boom-knuckles (2)
Это нужно было добавлять на первое апреля. Делать такое на постоянку - явно плохая идея и слишком кринж.

@Helm4142
Copy link
Author

@Helm4142 Ты не обновил медиа после убавления громкости. Сделать бы, что бы ревьюер смог оценить

Буэээ... А как?

@Helm4142
Copy link
Author

@Kit0vras А эт для чего?

@Kit0vras
Copy link

@Kit0vras А эт для чего?

Первое апреля уже закончилось, месяц назад

@Helm4142
Copy link
Author

@Kit0vras А эт для чего?

Первое апреля уже закончилось, месяц назад

Так это мем, смешной звук для атмосов.

@Helm4142
Copy link
Author

Я не задумывал это как прикол на 1 апреля

@Helm4142
Copy link
Author

Я куда-то понатыкал...

@Helm4142
Copy link
Author

И как фиксить эту ошибку?

@DexlerXD
Copy link

?
у тебя всё нормально

@Helm4142
Copy link
Author

? у тебя всё нормально

Спасибо, думал опять фиксить придётся

@DexlerXD
Copy link

@Ady4ik я закрываю пр? Я с Кито против, такое максимум на первое апреля добавлять.

@Ady4ik
Copy link

Ady4ik commented Apr 29, 2024

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

@Helm4142
Copy link
Author

Насчёт тише могу сделать, а вот эффект эхо никак не получается, увы...

@Helm4142 Helm4142 closed this Apr 30, 2024
@Helm4142 Helm4142 deleted the PipeSound branch May 3, 2024 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants