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

Как не сыграть в ящик? #3134

Draft
wants to merge 12 commits into
base: dev-sierra
Choose a base branch
from
Draft

Как не сыграть в ящик? #3134

wants to merge 12 commits into from

Conversation

ImJustKisik
Copy link

@ImJustKisik ImJustKisik commented Jan 20, 2025 •
edited
Loading

### Здесь водятся драконы, а должно быть карго.

close: https://github.com/orgs/SierraBay/projects/2/views/1?pane=issue&itemId=21327618

[approx. 90,6% ready]

TODO:

  • Пересмотр всех цен на вещи
  • Доделать маяк
  • Пофиксить баги в консоли
  • Пофиксить рантаймы
  • Пофиксить ???
  • Заполнить маяки вещами
  • Молитва
  • Портировать ТГУИ
  • Родить интерфейс
  • ?????
  • ПРОФИТ!!!

image

Чейнджлог

🆑ВашНикнейм
bugfix: Пофиксил баг
wip: Добавил что-то, что ещё в процессе
tweak: Подправил что-то
soundadd: Добавил новый звук
sounddel: Удалил старый звук
rscadd: Добавил в игру что-то новое
rscdel: Удалил из игры что-то старое
imageadd: Добавил новые иконки и картинки
imagedel: Удалил старые иконки и картинки
maptweak: Отредактировал карту
spellcheck: Исправил опечатку
experiment: Добавил что-то экспериментальное
balance: Заребалансил что-то
admin: Сделал что-то с админскими тулзами
/🆑
  • Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
  • Я внимательно прочитал все свои изменения и багов в них не нашел.
  • Я запускал сервер со своими изменениями локально и все протестировал.

@github-actions github-actions bot added 🗺️ Карты Issue или PR затрагивает карты ⛰️ Спрайты PR затрагивает спрайты labels Jan 20, 2025
LordNest
LordNest reviewed Jan 22, 2025
View reviewed changes
code/modules/shuttles/shuttle_supply.dm Outdated
Copy link

Choose a reason for hiding this comment

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

Я не думаю что нам нужно трогать сапплай шаттл как таковой. Если на него не будет лендмарок и он будет удалён с ЦК и Сьерры, то он не будет мешать новой системе.

Copy link
Author

@ImJustKisik ImJustKisik Jan 22, 2025 •
edited by LordNest
Loading

Choose a reason for hiding this comment

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

Я не думаю что нам нужно трогать сапплай шаттл как таковой. Если на него не будет лендмарок и он будет удалён с ЦК и Сьерры, то он не будет мешать новой системе.

Я просто хочу ещё один шаттлик, чтобы на нём летали - это будет овермап активность, плюс прикольно, плюс каргонцы не такие бесполезные, плюс их можно ограбить по пути

LN: Лол, я могу комментить ответы. Зачем? А чёрт знает

Copy link

Choose a reason for hiding this comment

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

Я имею ввиду что его не нужно комментить в коде. Так-то я сделаю шаттл. Туда нужно поставить конструкцию типа телепортера, да?

LordNest
LordNest reviewed Jan 22, 2025
View reviewed changes
maps/mapsystem/maps.dm
@@ -473,14 +479,15 @@ var/global/const/MAP_HAS_RANK = 2 //Rank system, also togglable

/datum/map/proc/setup_economy()
for (var/datum/feed_network/N in news_network)
N.CreateFeedChannel("Nyx Daily", "SolGov Minister of Information", 1, 1)
N.CreateFeedChannel("Nyx Daily", "[company_short] Minister of Information", 1, 1) // [ERIS MODPACK EDIT]
Copy link

Choose a reason for hiding this comment

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

Это будет получается NT Minister of Information. Тогда уж NT Head of External Relations

LordNest
LordNest reviewed Jan 22, 2025
View reviewed changes
packs/0_eris_cargo/trading_stations/_faction.dm Outdated
Copy link

Choose a reason for hiding this comment

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

Предлагаю выделить 3-4 основных гарантированных маяка и 2-3 маяка на доспавн.

Условно нужен маяк НТ, где будут почти дублироваться наборы из грузового для корпоративных нужд (мелкое оружие, одежда) и ресурсы для починки (типа базовой стали/стекала) для судов НТ

Пару "независимых" с базовым эквипом в других направлениях

Солгововский с экипировкой от Гефеста.

В "негарантированные" предлагаю
Хеллтек и экипировку ГКК
ДАИС с ботами и прочим хайтек
Скреллов каких-нибудь.

Отдельным ПРом увеличу полезность обычного торгаша для розничной торговли + расставлю маяков.

Copy link
Author

Choose a reason for hiding this comment

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

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

Copy link

Choose a reason for hiding this comment

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

А может просто коэффициент вставить типа 0,25 от стоимости при продаже через маяк? :clueless:

Copy link
Author

Choose a reason for hiding this comment

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LordNest LordNest LordNest left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
⛰️ Спрайты PR затрагивает спрайты 🗺️ Карты Issue или PR затрагивает карты
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ImJustKisik @LordNest