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

Fixes engineering access #1614

Closed
wants to merge 1 commit into from

Conversation

kyunkyunkyun
Copy link

@kyunkyunkyun kyunkyunkyun commented Oct 24, 2024

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

Чинит доступы в рамках инженерного отдела.

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

Атмосферный отдел - атмосферный доступ (не констракшн, как на Мете, например) и т.д.
Сами инженеры не потеряют никуда доступ, т.к. у нас у всех должностей в этом отделе одинаковые доступы (кроме атмосов, они ущемлены в правах), так что это повлияет в большинстве случаев на тех, кто не относится к инженерному отделу. Например, БЩ больше не сможет бегать по всему атмосу на Цереброне, а РД проникать в инженерный отдел через спутник. Гравген вновь принадлежит СЕ, а не РД в том числе.

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

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

Changelog

🆑
tweak: Изменены доступы на шлюзах в пределах инженерного отдела для соответствия действительности.
/:cl:

@github-actions github-actions bot added 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation) Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) Цереброн Этот ПР изменил что-то на Цереброне (MetaStation) labels Oct 24, 2024
@m-dzianishchyts
Copy link
Collaborator

нападай айлонг нападай, сбоку заходи!

Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

Я отказываюсь это проверять, пока дифф бот не будет показывать только заденутую область, а не всю карту
Увы, но с таким дифф ботом это нереально отревьювить не потратив несколько часов минимум
Раздели мзенения доступов на несколько ПРов, фикс телепортерной на отдельный, изменения в инженерке/атмосе - тут

image

@AyIong AyIong added the Do Not Merge Не заливать. В ожидании хэда/хоста. Или просто временный тестмерж. label Nov 1, 2024
@github-actions github-actions bot added the Merge Conflict Ну блять... label Nov 9, 2024
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Nov 14, 2024
@kyunkyunkyun kyunkyunkyun changed the title Fixes and tweaks engineering access on maps Fixes engineering access Nov 14, 2024
Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

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

Показывает ок. Мапдиф чист

@PhantornRU PhantornRU requested a review from AyIong November 18, 2024 10:44
Copy link

github-actions bot commented Dec 3, 2024

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

@github-actions github-actions bot added the Stale ПР долго был не активен, и требует обновления. label Dec 3, 2024
Copy link

This pull request has not received any updates since being marked stale, and as such is now being automatically closed. Please feel free to re-open this pull request or open a new one once you have new updates.

@github-actions github-actions bot closed this Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Не заливать. В ожидании хэда/хоста. Или просто временный тестмерж. 📜 CL валиден Stale ПР долго был не активен, и требует обновления. 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation) Цереброн Этот ПР изменил что-то на Цереброне (MetaStation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants