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

Мелочи для админов #204

Merged
merged 4 commits into from
Dec 26, 2024

Conversation

Vonsant
Copy link
Collaborator

@Vonsant Vonsant commented Dec 26, 2024

Описание PR

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

Почему / Баланс

Просто для упрощения некоторых вещей админам. А погоду можно и для карт использовать.

Технические детали

Добавлена отдельная папка - Admeme. Туда будут сыпаться все админ-штуки, доступные для спавна только админам. Создана по той причине, что прототипов погоды как таковых не существует, а далее там будут всякие спецэффекты и прочее визуальное. Телепортер перемещен туда по той же причине, чтобы четко разделить адмемские штуки.

Медиа

image

Список изменений

🆑

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

@github-actions github-actions bot added size/M Changes: Localization Изменяет локализацию Changes: No C# ПР без сишарпа S: Untriaged labels Dec 26, 2024
@FireNameFN
Copy link
Collaborator

Переименуй в Admin.

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Переименуй в Admin.

Admeme это описание всех админ предметов в игре, официальное.

Copy link
Collaborator

@FireNameFN FireNameFN left a comment

Choose a reason for hiding this comment

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

Убери везде приписке CN. Мы их не используем.

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Убери везде приписке CN. Мы их не используем.

Опять же нет по причине того что может быть добавлен обычный Admeme портал в будушем.

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Оффами тойбишь.

@FireNameFN
Copy link
Collaborator

Мы не дописываем к прототипам CN. Если оффы добавят, то линтер покажет, что надо убрать.

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Могу у погоды удалить, её точно не добавят.

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Мы не дописываем к прототипам CN. Если оффы добавят, то линтер покажет, что надо убрать.

Я пишу прототипы джва года, и я тебе говорю, что это надо оставить с припиской. Портал точно. Потому что Admeme портал уже есть.

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Он просто не телепортирует между картами, только между гридами.

@FireNameFN
Copy link
Collaborator

Ладно, но почему приписка после, а не перед, как у CE?

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Мы не дописываем к прототипам CN. Если оффы добавят, то линтер покажет, что надо убрать.

Вот, существующий HandTeleporterAdmeme

image

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Ладно, но почему приписка после, а не перед, как у CE?

Потому что в начале всегда пишется тип прототипа и у каждого айди есть четкая база, как он должен быть написан. Что такое CE не ебу.

@FireNameFN
Copy link
Collaborator

Crystall Edge, проекта Эда. Раньше в сокращении назывался цп.

@Vonsant
Copy link
Collaborator Author

Vonsant commented Dec 26, 2024

Crystall Edge, проекта Эда. Раньше в сокращении назывался цп.

Безпон, значит ему так удобнее. Он давно закрыл репу

@FireNameFN FireNameFN merged commit e8a92f9 into space-syndicate:master Dec 26, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменяет локализацию Changes: No C# ПР без сишарпа S: Untriaged size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants