rework: Syndicate affiliates 2.0 #5932

Anorak2024 · 2024-09-23T07:39:19Z

Описание

Подрядчики :badguy:

Автор ТЗ которое отдаленно напоминает ПР - SanR
Куча кода взято от @ROdenFL. Огромное ему спасибо.
Спрайты от vadim722.
Кучу ошибок в текстах поисправлял @Ssheya
Кодстайл правит @Vladisvell
Часть идей для предметов Krovososko

Ссылка на предложение

https://discord.com/channels/617003227182792704/755125334097133628/1288788680776876032

Демонстрация изменений

Тесты

code/modules/antagonists/traitor/affiliates/_affiliates.dm

ROdenFL · 2024-09-24T08:16:08Z

Хоть написал чей код используешь

Anorak2024 · 2024-09-24T08:59:34Z

Хоть написал чей код используешь

Теперь написал

ROdenFL · 2024-09-24T09:01:28Z

Хоть написал чей код используешь

Теперь написал

Собаку перед ником поставь

Ssheya

Немного насрал по русскому языку.

code/datums/uplink_item.dm

tgui/packages/tgui/interfaces/Uplink.js

Ssheya · 2024-09-24T21:34:15Z

а теперь после того как насрал пойду спать

Ssheya · 2024-09-25T06:45:24Z

Сам такой

Ssheya

русский язык

code/datums/uplink_item.dm

code/game/gamemodes/objective.dm

Ssheya

ещё капля русского могучего

code/datums/uplink_item.dm

Ssheya · 2024-09-25T13:43:00Z

🤓

Ssheya

Великий, могучий, русский язык.

code/datums/uplink_item.dm

code/game/gamemodes/objective.dm

code/modules/antagonists/traitor/affiliates/tiger.dm

code/game/gamemodes/objective.dm

Vladisvell

Замечания по стилю кода

code/game/gamemodes/malfunction/Malf_Modules.dm

code/game/gamemodes/miniantags/borer/borer.dm

code/game/gamemodes/objective.dm

code/modules/antagonists/traitor/affiliates/self.dm

code/modules/antagonists/traitor/affiliates/tiger.dm

code/modules/antagonists/traitor/datum_traitor.dm

code/modules/research/rdconsole.dm

DefaultMihail · 2024-09-27T21:45:07Z

Некоево "Мииш" призвали без пинга, а пришел я Badguy
Если голосование не прошло то хотяб ссылку на предложение, а то я не ебу что за реворк и что трогали
По "Syndicate affiliates" нахожу только ссылки на этот пулл

Anorak2024 · 2024-09-28T04:56:44Z

Некоево "Мииш" призвали без пинга, а пришел я Badguy Если голосование не прошло то хотяб ссылку на предложение, а то я не ебу что за реворк и что трогали По "Syndicate affiliates" нахожу только ссылки на этот пулл

Оно прошло недавно, прост я ссылку забыл добавить. Если коротко - разные фракции нанимателей в синдикате, со своими целями и механами. В ПРе куча чего отличается от ТЗ и если это работает лучше, то так и задумано.

DefaultMihail · 2024-09-28T13:08:55Z

Проверял на Anorak2024:syndicate_affiliates_2.0
Не проверял засчитывание выполненный целей/выдачу целей/специфичные для расс вещи(и так насрано достаточно + впадлу).

QA "Срать вечно" вещает:

При выдаче трейторки с определённым подрядчиком, если назать отмена, то будет трейтор с Unknown(минор, это в админ панели)
1.1. Там же traitor with CONCRET affiliate, что за бетон и что он там делает? Гугл переводчик сила, школа - 💀. Используйтё лучше DeepL, там конкретный становится не бетоном, а specific.
После выбора подрядчика через кнопку, кнопка остаётся
(Вкусовщина) Насрали текстом в выборе подрядчиков, тяжело будет понять плюсы/минусы того или иного подрядчика. В аплинке так же насрано
Нет иконок у многих подрядчиков.
У MI13:
5.1. По ТЗ доступна к покупке только сидни ева - ноуп, можно купить всё, ограничена только вкладка из п.1 ТЗ,
5.2. По ТЗ имеют 50ТК, при выборе через кнопку и через панель - 20ТК
5.3. По ТЗ(снизу требования к разработке) противоречим 5.1 "Невозможно купить Syndicate Hardsuit." - его там реально нет, но ТЗ говна получается
Cybersun Industries:
6.1. Текст скидок кривой и выбивается (Актуально для всех подрядчиков, где выдаются скидки):

6.2. Invasive Beacon даёт 6 Data 3 Bio(должно было быть Magnets) 1 Illegal
Hematogenic Industries:
7.1. Экстракт стоит 74ТК, по ТЗ - 0ТК. В том же ТЗ цена 60ТК. Чё за хуйня?
7.2. Цели после становлением вампиром не изменяются, при использовании на другую куклу не добавляются, получается вампир с пустыми целями
7.3. Никаких стадий превращения не замечено
Gorlex Marauders:
8.1. Таймер на 20 минут работает не от начала раунда, а с момента выбора
8.2. Не видят других оперативников Gorlex, худа нет
SELF:
9.1. При использовании на киборге синего емага законы становятся пустыми, а не предопределёнными
9.2. Ничего не мешает юзать емаг/плату
Tiger Cooperative:
10.1. Могут купить дигл
10.2. Могут пользоваться любым дальнобойным оружеем
10.3. Скидки не обнаружены
10.4. Где egg implanter?

ROdenFL · 2024-09-30T07:51:03Z

Нахуя вы Аплинк перевели, изверги

NightDawnFox · 2024-09-30T08:13:53Z

Тебя забыли спросить, иди на оффпару кодь

Den109G

Раз перевод - лучше через семь кругов перевода пустить ПР.

Anorak2024 · 2024-10-01T09:32:34Z

Ну, раз уж плашку повесили, терять нечего

code/datums/uplink_item.dm

Anorak2024 · 2024-10-02T16:48:31Z

Проверял на Anorak2024:syndicate_affiliates_2.0 Не проверял засчитывание выполненный целей/выдачу целей/специфичные для расс вещи(и так насрано достаточно + впадлу).

QA "Срать вечно" вещает:

При выдаче трейторки с определённым подрядчиком, если назать отмена, то будет трейтор с Unknown(минор, это в админ панели)
1.1. Там же traitor with CONCRET affiliate, что за бетон и что он там делает? Гугл переводчик сила, школа - 💀. Используйтё лучше DeepL, там конкретный становится не бетоном, а specific.

После выбора подрядчика через кнопку, кнопка остаётся

(Вкусовщина) Насрали текстом в выборе подрядчиков, тяжело будет понять плюсы/минусы того или иного подрядчика. В аплинке так же насрано

Нет иконок у многих подрядчиков.

У MI13:
5.1. По ТЗ доступна к покупке только сидни ева - ноуп, можно купить всё, ограничена только вкладка из п.1 ТЗ,
5.2. По ТЗ имеют 50ТК, при выборе через кнопку и через панель - 20ТК
5.3. По ТЗ(снизу требования к разработке) противоречим 5.1 "Невозможно купить Syndicate Hardsuit." - его там реально нет, но ТЗ говна получается

Cybersun Industries:
6.1. Текст скидок кривой и выбивается (Актуально для всех подрядчиков, где выдаются скидки):

6.2. Invasive Beacon даёт 6 Data 3 Bio(должно было быть Magnets) 1 Illegal

Hematogenic Industries:
7.1. Экстракт стоит 74ТК, по ТЗ - 0ТК. В том же ТЗ цена 60ТК. Чё за хуйня?
7.2. Цели после становлением вампиром не изменяются, при использовании на другую куклу не добавляются, получается вампир с пустыми целями
7.3. Никаких стадий превращения не замечено

Gorlex Marauders:
8.1. Таймер на 20 минут работает не от начала раунда, а с момента выбора
8.2. Не видят других оперативников Gorlex, худа нет

SELF:
9.1. При использовании на киборге синего емага законы становятся пустыми, а не предопределёнными
9.2. Ничего не мешает юзать емаг/плату

Tiger Cooperative:
10.1. Могут купить дигл
10.2. Могут пользоваться любым дальнобойным оружеем
10.3. Скидки не обнаружены
10.4. Где egg implanter?

Была частично фича. Если не выбрать самому, выбирался старый тритор, но сделал чтоб шла отмена выбора.
1.1 Поменял
Пофиксил
В выборе оставил плюсы, минусы и типовые цели.
Теперь иконок нет у всех подрячиков)))0)
5.1/5.2 Думаю что автор имел ввиду что из ригов доступна только синдиева. Оно щас так.
5.3. Фича, 50 ТК + бонд - много.
6.1 Больше не кривой
6.2 Исправил
7.1 Там тип обычный 74 (цена выше чем в ТЗ и это по плану), а так-же есть нужный для цели, который стоит 0.
7.2 Ща появляются.
7.3 Фича, решил что если сделать, будет чисто афк стояние минут 5 в дормах.
8.1 Вроде исправил
8.2 Пофиг, потом сделаю. Точно и смотрящий и тот на кого смотрят были горлексами с выданным аплинком?
9.1 Фича, странно освобождать борга от законов, только чтобы дать ему другие. С новым сводом законов он не мог пойти в рабство при желании, а ведь это его выбор.
9.2 Вроде пофиксил
10.1 Больше не могут. Какой-то гений поместил дигл в раздел бесполезного.
10.2 Ща вроде не могут
10.3 Теперь есть
10.4 Есть

DefaultMihail · 2024-10-02T17:10:23Z

На выходных только смогу глянуть, если ещё будет актуально(если не замёрджат) и если ещё нужно

Anorak2024 · 2024-10-02T17:11:23Z

На выходных только смогу глянуть, если ещё будет актуально(если не замёрджат) и если ещё нужно

Совсем не срочно и замержат не скоро.

github-actions bot added DMI Affects DMI files TGUI Affects TGUI javascript Pull requests that update Javascript code labels Sep 23, 2024

ROdenFL reviewed Sep 24, 2024

View reviewed changes

code/modules/antagonists/traitor/affiliates/_affiliates.dm Outdated Show resolved Hide resolved

Ssheya reviewed Sep 24, 2024

View reviewed changes

Ssheya reviewed Sep 25, 2024

View reviewed changes

code/datums/uplink_item.dm Outdated Show resolved Hide resolved

code/game/gamemodes/objective.dm Outdated Show resolved Hide resolved

code/game/gamemodes/objective.dm Outdated Show resolved Hide resolved

Ssheya reviewed Sep 25, 2024

View reviewed changes

code/datums/uplink_item.dm Outdated Show resolved Hide resolved

Ssheya reviewed Sep 26, 2024

View reviewed changes

code/datums/uplink_item.dm Outdated Show resolved Hide resolved

code/datums/uplink_item.dm Outdated Show resolved Hide resolved

code/game/gamemodes/objective.dm Outdated Show resolved Hide resolved

code/modules/antagonists/traitor/affiliates/tiger.dm Outdated Show resolved Hide resolved

Ssheya reviewed Sep 26, 2024

View reviewed changes

code/game/gamemodes/objective.dm Outdated Show resolved Hide resolved

Vladisvell requested changes Sep 27, 2024

View reviewed changes

Anorak2024 changed the title ~~Syndicate affiliates 2.0~~ rework: Syndicate affiliates 2.0 Sep 27, 2024

Den109G added the перевод Съешь же ещё этих мягких французских булок, да выпей чаю label Sep 30, 2024

Den109G requested changes Sep 30, 2024

View reviewed changes

Den109G reviewed Oct 2, 2024

View reviewed changes

github-actions bot added the Merge Conflict This doesn't seem right label Oct 3, 2024

Anorak2024 marked this pull request as ready for review October 7, 2024 19:44

Anorak2024 requested a review from Bizzonium as a code owner October 7, 2024 19:44

Anorak2024 closed this Oct 7, 2024

Anorak2024 force-pushed the syndicate_affiliates_2.0 branch from f6efa8b to 01267f6 Compare October 7, 2024 20:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

rework: Syndicate affiliates 2.0 #5932

rework: Syndicate affiliates 2.0 #5932

Anorak2024 commented Sep 23, 2024 •

edited

Loading

ROdenFL commented Sep 24, 2024

Anorak2024 commented Sep 24, 2024

ROdenFL commented Sep 24, 2024

Ssheya left a comment

Ssheya commented Sep 24, 2024

Ssheya commented Sep 25, 2024

Ssheya left a comment

Ssheya left a comment

Ssheya commented Sep 25, 2024

Ssheya left a comment

Vladisvell left a comment

DefaultMihail commented Sep 27, 2024

Anorak2024 commented Sep 28, 2024 •

edited

Loading

DefaultMihail commented Sep 28, 2024

ROdenFL commented Sep 30, 2024

NightDawnFox commented Sep 30, 2024

Den109G left a comment •

edited

Loading

Anorak2024 commented Oct 1, 2024

Anorak2024 commented Oct 2, 2024

DefaultMihail commented Oct 2, 2024

Anorak2024 commented Oct 2, 2024

rework: Syndicate affiliates 2.0 #5932

rework: Syndicate affiliates 2.0 #5932

Conversation

Anorak2024 commented Sep 23, 2024 • edited Loading

Описание

Ссылка на предложение

Демонстрация изменений

Тесты

ROdenFL commented Sep 24, 2024

Anorak2024 commented Sep 24, 2024

ROdenFL commented Sep 24, 2024

Ssheya left a comment

Choose a reason for hiding this comment

Ssheya commented Sep 24, 2024

Ssheya commented Sep 25, 2024

Ssheya left a comment

Choose a reason for hiding this comment

Ssheya left a comment

Choose a reason for hiding this comment

Ssheya commented Sep 25, 2024

Ssheya left a comment

Choose a reason for hiding this comment

Vladisvell left a comment

Choose a reason for hiding this comment

DefaultMihail commented Sep 27, 2024

Anorak2024 commented Sep 28, 2024 • edited Loading

DefaultMihail commented Sep 28, 2024

ROdenFL commented Sep 30, 2024

NightDawnFox commented Sep 30, 2024

Den109G left a comment • edited Loading

Choose a reason for hiding this comment

Anorak2024 commented Oct 1, 2024

Anorak2024 commented Oct 2, 2024

DefaultMihail commented Oct 2, 2024

Anorak2024 commented Oct 2, 2024

Anorak2024 commented Sep 23, 2024 •

edited

Loading

Anorak2024 commented Sep 28, 2024 •

edited

Loading

Den109G left a comment •

edited

Loading