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

Зимняя карта!1 #6478

Closed
wants to merge 3 commits into from

Conversation

4310V343k
Copy link
Member

@4310V343k 4310V343k commented Dec 5, 2020

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

Ура! Гирляндочки, ёлочка, НОВЫЙ ГОД!

За лесом сюда: #6499
Подробнее о фичах: #2917

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

Атмосферность

Авторство

BesumniiChebureck

Чеинжлог

🆑

  • map: Зимняя карта на время нового года!

@TauKitty
Copy link
Contributor

TauKitty commented Dec 5, 2020

Changelog status: ✔️

@Kortez90
Copy link
Contributor

Kortez90 commented Dec 5, 2020

Почему бы сразу не привязать переключение зимней карты ко времени?

@4310V343k
Copy link
Member Author

4310V343k commented Dec 5, 2020

Почему бы сразу не привязать переключение зимней карты ко времени?

Привяжи

И сделай так, что бы гирлянды/украшения сами ставились каждый раз, когда кто-то делает изменения в карте

@Kortez90
Copy link
Contributor

Kortez90 commented Dec 5, 2020

Твой же ПР.
Сделай копию карты и включай ее, когда наступает новый год. Все изменения на ней отдельные.

@4310V343k
Copy link
Member Author

Прикольно будет на новый год включать карту годовой давности, ога

@Kortez90
Copy link
Contributor

Kortez90 commented Dec 5, 2020

Что-то мешает ее обновлять вместе с оригиналом?
Хотя если дело только в гирляндах, то копия и не нужна.

@Sakuya-Izayoi
Copy link
Contributor

Что-то мешает ее обновлять вместе с оригиналом?

то что это в два раза больше работы и про это легко забыть

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

@4310V343k
Copy link
Member Author

Что-то мешает ее обновлять вместе с оригиналом?

Лень обновляторов

Хотя если дело только в гирляндах, то копия и не нужна.

Вот сам и делай прикольную генерацию гирлянд, мне-то это зачем?

@TauKitty
Copy link
Contributor

TauKitty commented Dec 5, 2020

This pull request has been mentioned on Tau Ceti Station Forum. There might be relevant details there:

https://forum.taucetistation.org/t/vozvrashhenie-zimnego-lesa/14446/72

@4310V343k 4310V343k force-pushed the newyear2021 branch 2 times, most recently from e78509d to 44a606c Compare December 5, 2020 14:26
@atik1n
Copy link
Member

atik1n commented Dec 5, 2020

Я конечно не эксперт, но мне кажется именно новогоднюю карту надо ближе к НГ, за неделю-полторы + пару дней после. А вот зимнюю карту как лес в прошлом году надо бы портировать пораньше

@4310V343k
Copy link
Member Author

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

@volas
Copy link
Member

volas commented Dec 6, 2020

Комментатор на два этажа выше прав, гирлянды вносят изменения по всей карте и это легко конфликтует с каждым вторым мап-пром.

И ты не пробовал разобраться с лесом из #4417? Там может быть проблема с оверлеем снегопада, но в остальном всё должно быть так же.

@Kortez90
Copy link
Contributor

Kortez90 commented Dec 6, 2020

Так почему бы не автоматизировать процесс, чтобы каждый раз не заниматься им вручную?
Можно использовать ландмарки, например.
Или зоны, в которых можно искать окна и на каждое вешать по гирлянде.

@4310V343k
Copy link
Member Author

Так почему бы не автоматизировать процесс, чтобы каждый раз не заниматься им вручную?
Можно использовать ландмарки, например.
Или зоны, в которых можно искать окна и на каждое вешать по гирлянде.

Ну сделай! Конечно, твой крутой скрипт будет использоваться раз в год, но зато никому больше не надо будет тратить полчаса своей жизни на это

@UsadoSmokk
Copy link
Contributor

Я один ёлку ставлю уже в 10 числах? Просто не знаю, для меня привычны уже эти все украшения, вот в Омске у нас, допустим, ТЦ ёлку ещё 27 ноября поставили и украсили. Посему, а чеб нет? Единственное, может все таки подумать про входы и выходы, чтобы комфортно было всем, что для гулен, что для "домоседов"?

@SomeUselessGuy
Copy link
Contributor

SomeUselessGuy commented Dec 6, 2020

А можно я, в лице говнокодера, буду давать свои говносоветы?
Можно же в init() стекла засунуть установку оверлея гирлянд с каким-то шансом
Капецкакялюблюоверлеиодаааааааа

@4310V343k
Copy link
Member Author

4310V343k commented Dec 23, 2020

Блять
коммиты потекли

@4310V343k
Copy link
Member Author

Я тебе мержну потом по голове, пр не доделан ещё

Co-authored-by: BesumniiChebureck <BesumniiChebureck@users.noreply.github.com>
@4310V343k 4310V343k marked this pull request as ready for review December 24, 2020 14:16
@T6751 T6751 added Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже and removed Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже labels Dec 25, 2020
@4310V343k
Copy link
Member Author

4310V343k commented Dec 25, 2020

@T6751 можно в тестмерж вместе с #6499
При условии того, что оно пройдёт через мапхук
image

@4310V343k 4310V343k marked this pull request as draft December 25, 2020 10:34
@4310V343k
Copy link
Member Author

#6499

@T6751 T6751 removed the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Dec 25, 2020
@stale
Copy link

stale bot commented Jan 10, 2021

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@stale stale bot added the Stalled PR label Jan 10, 2021
@T6751 T6751 closed this Jan 10, 2021
@4310V343k 4310V343k deleted the newyear2021 branch May 29, 2021 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants