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

Prefs cleanup (cid history), admin stuff #13212

Merged
merged 6 commits into from
Jun 26, 2024
Merged

Conversation

volas
Copy link
Member

@volas volas commented Jun 25, 2024

Описание изменений

Вынес часть из #13141, чтоб его не перегружать.

Сейвфайлы игроков больше не используются для хранения cid-ов, у нас уже давно есть база данных для этого. ignore_cid_warning вообще никогда не использовался.

Пока всё равно был тут, оформил админам две новые панельки для истории игроков. И сделал guard датум более постоянным, а то я устал наступать на грабли с бьендовским клиентом, которому ничего нельзя доверить.

Почему и что этот ПР улучшит

Авторство

Чеинжлог

@TauKitty
Copy link
Contributor

Changelog status: 🔘 No changelog

Comment on lines -351 to -352
S["cid_list"] >> cid_list
S["ignore_cid_warning"] >> ignore_cid_warning
Copy link
Member Author

Choose a reason for hiding this comment

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

я пока не чищу старые записи, со всей этой работой с #13141 планирую использовать их как бэкап. Потом просто все sav файлы с сервера можно будет потереть.

@volas volas changed the title Prefs cleanup, admin stuff Prefs cleanup (cid history), admin stuff Jun 25, 2024
@AzzyDreemurr13 AzzyDreemurr13 added the Admin Всё что касается админов. Логирование, инструментарий и т.д. label Jun 25, 2024
@volas volas merged commit 415bca4 into TauCetiStation:master Jun 26, 2024
12 checks passed
@volas volas mentioned this pull request Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Всё что касается админов. Логирование, инструментарий и т.д. Preferences Update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants