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

bugfix/admin: vv options rights fix #6378

Merged

Conversation

dageavtobusnick
Copy link
Contributor

@dageavtobusnick dageavtobusnick commented Jan 18, 2025

Описание

Исправляет ошибки, связанные с доступом к различным возможностям вв.
Данный ПР убирает общий ограничитель топика на какие-либо флаги, за ненадобностью. Так же закрывает редактирование листов на R_VAREDIT (что вполне логично) и редактирование трейтов на R_DEBUG|R_ADMIN , что тоже логично, ибо у триалов эта опция была давно, а для дебага она крайне полезна. Все остальные не закрытые опции являются опциями просмотра и итак ограничены доступом в VV.

Причина создания ПР / Почему это хорошо для игры

Дело в том, что создатель данного пр #5677 для того, чтобы у контрибуторов была возможность редактировать переменные в VV привязал общий доступ к топику VV к R_VAREDIT. Проблема в том, что VV это не только варэдит, но и многие другие опции. Каждая из опций закрыта на свой флаг(кроме парочки). Из-за даного изменения например у триалов пропали некоторые кнопки, которые у них до изменения были и, судя по описанию пр-а, идеи забирать их не было. Данный пр убирает ненужное общее ограничение и закрывает опции, которые должны были быть закрыты(но почему-то не закрыты).

Тесты

Не нужны. Да и их трудновато проводить на локалке с фул доступом.

@BeebBeebBoob BeebBeebBoob merged commit d813647 into ss220-space:master220 Jan 18, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants