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

[TM] Make credits toggleable #1431

Merged
merged 10 commits into from
Aug 23, 2024

Conversation

m-dzianishchyts
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts commented Aug 14, 2024

Что этот PR делает

Добавляет возможность отключить автоматическое воспроизведение титров в настройках игры. Администратор все равно может зафорсить титры в текущем раунде, если считает нужным.
Добавляет новую таблицу с префами, надо будет так и другие фичи сделать.

Почему это хорошо для игры

Можно не прожимать "Прекратить смотреть синематик" каждый раунд.

Изображения изменений

image

Тестирование

  • Свойство в настройках реально влияет на воспроизведение титров.
  • Админ реально может зафорсить титры, никакие настройки от этого не спасут.
  • Настройки сохранились в тестовой БД и подсосались после перезапуска сервера.

Changelog

🆑 Maxiemar
add: Автоматическое воспроизведение титров в конце раунда теперь можно отключить в настройках игры.
add: Администрация может зафорсить титры на конец раунда, если захочет всем напомнить, как они выглядят. Server -> Toggle Credits.
/:cl:

@github-actions github-actions bot added :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги. 💾 Изменение конфига Ф-ф-фуриор...? 💿 SQL Надо править БД. labels Aug 14, 2024
// SS220 ADDITION - START
if(PREFTOGGLE_TOGGLE220)
dat += "<td style='width: 20%'><a href='byond://?_src_=prefs;preference=preference_toggles;toggle=[toggle.UID()];'>[(toggles220 & toggle.preftoggle_bitflag) ? "<span class='good'>Enabled</span>" : "<span class='bad'>Disabled</span>"]</a></td>"
// SS220 ADDITION - END
Copy link
Collaborator

Choose a reason for hiding this comment

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

Надо будет придумать что-то дабы не влезать в кор ради префов

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Здесь поражение, остается слезно молить о тотальном рефакторе в рамках перехода на тгуи

@m-dzianishchyts m-dzianishchyts marked this pull request as draft August 15, 2024 00:08
@github-actions github-actions bot added the Merge Conflict Ну блять... label Aug 15, 2024
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Aug 20, 2024
@m-dzianishchyts m-dzianishchyts force-pushed the toggle-credits-pref-feat branch from 51a470a to 5c6d441 Compare August 20, 2024 10:00
@m-dzianishchyts m-dzianishchyts marked this pull request as ready for review August 20, 2024 11:46
@m-dzianishchyts m-dzianishchyts changed the title Make credits toggleable [TM] Make credits toggleable Aug 20, 2024
Copy link

@Furrior Furrior left a comment

Choose a reason for hiding this comment

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

Вроде, все норм, но я только проснулся, так что мог просмотреть

modular_ss220/credits/_credits.dm Outdated Show resolved Hide resolved
@m-dzianishchyts
Copy link
Collaborator Author

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

@Legendaxe Legendaxe merged commit 4d7d93a into ss220club:master Aug 23, 2024
13 checks passed
Hearstrings pushed a commit to Hearstrings/Paradise-SS220 that referenced this pull request Aug 23, 2024
## Что этот PR делает
Добавляет возможность отключить автоматическое воспроизведение титров в
настройках игры. Администратор все равно может зафорсить титры в текущем
раунде, если считает нужным.
Добавляет ***новую таблицу с префами***, надо будет так и другие фичи
сделать.

## Почему это хорошо для игры
Можно не прожимать "Прекратить  смотреть синематик" каждый раунд.

## Изображения изменений


![image](https://github.com/user-attachments/assets/e50fbe28-ee32-4304-a16c-c8c6b73b77b5)


## Тестирование
- [x] Свойство в настройках реально влияет на воспроизведение титров.
- [x] Админ реально может зафорсить титры, никакие настройки от этого не
спасут.
- [x] Настройки сохранились в тестовой БД и подсосались после
перезапуска сервера.

## Changelog

:cl: Maxiemar
add: Автоматическое воспроизведение титров в конце раунда теперь можно
отключить в настройках игры.
add: Администрация может зафорсить титры на конец раунда, если захочет
всем напомнить, как они выглядят. Server -> Toggle Credits.
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💿 SQL Надо править БД. 💾 Изменение конфига Ф-ф-фуриор...? 📜 CL валиден :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants