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.

Sign up for GitHub

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

Jump to bottom

[Prime-Only] Add: Prime Event outfits&clothes #1647

Closed
wants to merge 16 commits into from
Closed

[Prime-Only] Add: Prime Event outfits&clothes #1647

wants to merge 16 commits into from

Conversation

konushi
Copy link

@konushi konushi commented Nov 10, 2024

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

Вернул праймовские аутфиты для ивентов. Меньше возиться с vv.

Жду спрайты оставшихся шмоток.

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

Админы жмут меньше кнопок - это хорошо.

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

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

Локалка, ворк.

Changelog

🆑
add: [PRIME ONLY] вернул праймовские аутфиты. Админы меньше трогают кнопок - игроки счастливей.
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Nov 10, 2024
@konushi konushi marked this pull request as draft November 10, 2024 00:12
@Furrior
Copy link

Furrior commented Nov 10, 2024 •
edited
Loading

А че их просто не вмержить на постоянку и не только в прайм?

@konushi
Copy link
Author

konushi commented Nov 10, 2024

А че их просто не вмержить на постоянку и не только в прайм?

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

@github-actions github-actions bot added the 🔉 Звук Услада для ушей. label Nov 10, 2024
@konushi konushi marked this pull request as ready for review November 10, 2024 19:15
@m-dzianishchyts
Copy link
Collaborator

Надо обезличить аутфиты, чтобы не было собственных имен.
Один адмеместратор уже неактуален

@konushi
Copy link
Author

konushi commented Nov 10, 2024

Надо обезличить аутфиты, чтобы не было собственных имен. Один адмеместратор уже неактуален

В плане? В выборе эквипмента они идут с наименованием из "name". Собственные имена только в датуме

@m-dzianishchyts
Copy link
Collaborator

В плане? В выборе эквипмента они идут с наименованием из "name". Собственные имена только в датуме

Все собственные имена из кода надо почистить, оставить только то, что крепко в лоре отражается. Вместо имен надо коды какие-то типа alt1, alt2. Если кого-то потом ОТМЕНЯТ, не хочется упоминания чистить

@konushi
Copy link
Author

konushi commented Nov 10, 2024

В плане? В выборе эквипмента они идут с наименованием из "name". Собственные имена только в датуме

Все собственные имена из кода надо почистить, оставить только то, что крепко в лоре отражается. Вместо имен надо коды какие-то типа alt1, alt2. Если кого-то потом ОТМЕНЯТ, не хочется упоминания чистить

Ща поменяю. Никого никогда не отменяли, не понимаю о чем ты jokerge

@m-dzianishchyts m-dzianishchyts self-requested a review November 10, 2024 20:51
@konushi konushi marked this pull request as draft November 11, 2024 16:26
@konushi
Copy link
Author

konushi commented Nov 11, 2024

жду иконки

m-dzianishchyts
m-dzianishchyts requested changes Nov 13, 2024
View reviewed changes
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts 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/prime_only/code/outfit.dm Show resolved Hide resolved
modular_ss220/prime_only/code/clothing/suits.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/clothing/suits.dm
)
var/datum/action/item_action/chameleon/stealth/stealth_action

/datum/action/item_action/chameleon/stealth
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

Choose a reason for hiding this comment

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

Можно еще с пальто в руке использовать

modular_ss220/prime_only/code/clothing/suits.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/clothing/suits.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/clothing/head.dm Show resolved Hide resolved
modular_ss220/prime_only/code/clothing/suits.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/clothing/under.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/outfit.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/spells.dm Outdated Show resolved Hide resolved
@konushi
Copy link
Author

konushi commented Nov 13, 2024

там НЕОЖИДАННО еще пара спрайтов будет, так что спим дальше :mimir

m-dzianishchyts
m-dzianishchyts requested changes Nov 13, 2024
View reviewed changes
modular_ss220/prime_only/code/clothing/mask.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/outfit.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/outfit.dm Outdated Show resolved Hide resolved
modular_ss220/prime_only/code/outfit.dm Outdated Show resolved Hide resolved
m-dzianishchyts
m-dzianishchyts approved these changes Nov 13, 2024
View reviewed changes
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts left a comment

Choose a reason for hiding this comment

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

А я поверил :aga:

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 1, 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 Stale ПР долго был не активен, и требует обновления. Merge Conflict Ну блять... labels Dec 1, 2024
@m-dzianishchyts
Copy link
Collaborator

m-dzianishchyts commented Dec 3, 2024 •
edited
Loading

Можно сохранить что угодно в базу и локально через сериализацию (доступно только добавившему).
По этой причине персональные аутфиты отсюда можно будет вырезать.

Debug -> Serialize Marked Datum (помечать через VV)
Debug -> Deserialize JSON datum (из буфера обмена)
Deubg -> Spawn Saved JSON Datum (из БД)

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

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 19, 2024
@github-actions GitHub Actions
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 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-dzianishchyts m-dzianishchyts m-dzianishchyts approved these changes

Assignees
No one assigned
Labels
Merge Conflict Ну блять... 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден 🔉 Звук Услада для ушей. Stale ПР долго был не активен, и требует обновления.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@konushi @Furrior @m-dzianishchyts @ss220bot @Fullonibus