Корпоративная одежда для глав.врача #2345

Tris0r · 2024-12-12T09:48:57Z

Описание PR

Добавление в лодаут ГВ (10 часов) и в комод ГВ корпоративного комбинезона и юбки-комбинезона

Медиа

Проверки

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

Изменения
🆑 Cortez, molochk

add: Добавлена корпоративная одежда в лодаут и в комод главного врача

github-actions · 2024-12-12T09:49:42Z

RSI Diff Bot; head commit 65b957f merging into 8c34022
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Clothing/Uniforms/Jumpskirt/cmo_corpW.rsi

State	Old	New	Status
equipped-INNERCLOTHING-monkey			Added
equipped-INNERCLOTHING			Added
icon			Added
inhand-left			Added
inhand-right			Added

Resources/Textures/SS220/Clothing/Uniforms/Jumpsuit/cmo_corp.rsi

State	Old	New	Status
equipped-INNERCLOTHING-monkey			Added
equipped-INNERCLOTHING			Added
icon			Added
inhand-left			Added
inhand-right			Added

Edit: diff updated after 65b957f

Tris0r · 2024-12-12T09:51:02Z

Это мой первый PR, поэтому будьте ко мне снисходительнее...
:gatoowo18:

stalengd · 2024-12-12T16:06:20Z

Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpsuit/jumpsuits.yml

+  id: ClothingUniformJumpsuitCMOCorporate
+  name: corporate CMO Jumpsuit
+  description: «A beautiful, stylish suit of a real chief physician. He emphasizes the nature of a person and his beauty of charisma...» There is nothing further on the unopened label.
+  suffix: For playing time


Suggested change

suffix: For playing time

suffix: For playtime

stalengd · 2024-12-12T16:09:38Z

Resources/Textures/SS220/Clothing/Uniforms/Jumpskirt/cmo_corpW.rsi/meta.json

@@ -0,0 +1,30 @@
+{
+  "version": 1,
+  "license": "CC-BY-SA-3.0",


Автор настоял на этой лицензии? Если так, то не проблема, но кто знает.

Автор наверное вообще не в курсе про лицензии.

Пихай тогда нашу дефолтную CLA, примеры найдёшь, думаю, почти каждый наш спрайт по ней

stalengd · 2024-12-12T16:13:27Z

Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpskirt/jumpskirts.yml

+  id: ClothingUniformJumpskirtCMOCorporate
+  name: corporate CMO Jumpskirt
+  description: «A beautiful, stylish suit of a real chief physician. He emphasizes the nature of a person and his beauty of charisma...» There is nothing further on the unopened label.
+  suffix: For playing time


Suggested change

suffix: For playing time

suffix: For playtime

stalengd · 2024-12-12T16:14:20Z

Resources/Prototypes/Catalog/Fills/Lockers/dressers.yml

+      - id: ClothingUniformJumpsuitCMOCorporate #SS220-CMOCorporate
+      - id: ClothingUniformJumpskirtCMOCorporate #SS220 - CMOCorporate


Тогда уже какой-то один вариант хотя бы в рамках одного файла

Suggested change

- id: ClothingUniformJumpsuitCMOCorporate #SS220-CMOCorporate

- id: ClothingUniformJumpskirtCMOCorporate #SS220 - CMOCorporate

- id: ClothingUniformJumpsuitCMOCorporate #SS220-CMOCorporate

- id: ClothingUniformJumpskirtCMOCorporate #SS220-CMOCorporate

А тут то что надо

пробелы убрать

stalengd · 2024-12-12T16:15:57Z

Resources/Locale/ru-RU/ss220/clothing/Jumpskirt/jumpskirt.ftl

@@ -31,3 +31,6 @@ ent-ClothingUniformJumpskirtProfessor = форма профессора
 ent-ClothingUniformJumpskirtWarehouseman = рабочий комбинезон-юбка заведующего складом
    .desc = Практичный, удобный, не стесняющий движения и главное стильный юбка-комбинезон. У вас с ним связаны плохие воспоминания о прошлом начальнике
    .suffix = За время в игре
+ent-ClothingUniformJumpskirtCMOCorporate = корпоративная юбка-комбинезон главного врача
+    .desc = "Красивый, стильный костюм настоящего главврача. Он подчёркивает натуру человека и его красоту харизмы..." Дальше на неоткрываемой этикетке ничего нет.


Может быть "красоту его харизмы"? И что такое "неоткрываемая этикетка"?

stalengd · 2024-12-12T16:17:19Z

Resources/Prototypes/Loadouts/Jobs/Medical/chief_medical_officer.yml

Закинуть изменения в нашу папку

Tris0r · 2024-12-12T16:47:18Z

Сказал же не душить...

EstKemran · 2024-12-12T17:33:49Z

Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpskirt/jumpskirts.yml

@@ -135,6 +135,19 @@

 # Medical

+  # Главный врач
+- type: entity
+  parent: ClothingUniformBase


Поменять на ClothingUniformSkirtBase

Why...

Без этого парента, на юбках, одетых на женских персонажах, если не ошибаюсь, пропадал пиксель. И чтобы этого бага избежать, создали базовый прототип ClothingUniformSkirtBase, где у компонента Clothing есть параметр femaleMask: UniformTop

lexaSvarshik

Немного по локализации, пометки исправить, в остальном нормально.
Но, честности ради, я бы переделал описание.

lexaSvarshik · 2024-12-12T19:12:22Z

Resources/Locale/ru-RU/ss220/clothing/Jumpskirt/jumpskirt.ftl

@@ -31,3 +31,6 @@ ent-ClothingUniformJumpskirtProfessor = форма профессора
 ent-ClothingUniformJumpskirtWarehouseman = рабочий комбинезон-юбка заведующего складом
    .desc = Практичный, удобный, не стесняющий движения и главное стильный юбка-комбинезон. У вас с ним связаны плохие воспоминания о прошлом начальнике
    .suffix = За время в игре
+ent-ClothingUniformJumpskirtCMOCorporate = корпоративная юбка-комбинезон главного врача
+    .desc = "Красивый, стильный костюм настоящего главврача. Он подчёркивает натуру человека и его красоту харизмы..." Дальше на неоткрываемой этикетке ничего нет.


"Человека" заменить на что-то более нейтральное, у нас кроме людей другие гуманоиды есть

Даже в таких контекстах "человек" обычно норм, всё таки у него смысл чуть шире чем просто раса.

Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpskirt/jumpskirts.yml

Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpsuit/jumpsuits.yml

lexaSvarshik · 2024-12-12T19:18:11Z

Resources/Prototypes/Loadouts/loadout_groups.yml

@@ -1378,6 +1378,8 @@
  - ChiefMedicalOfficerJumpskirt
  - ChiefMedicalOfficerTurtleneckJumpsuit
  - ChiefMedicalOfficerTurtleneckJumpskirt
+  - CMOCorpJumpsuit #ss220
+  - CMOCorpJumpskirt #ss220


Suggested change

- CMOCorpJumpskirt #ss220

- CMOCorpJumpsuit #SS220-CMOCorporate

Всё ещё не исправлено

Resources/Prototypes/Loadouts/loadout_groups.yml

…t/jumpskirts.yml Co-authored-by: Svarshik <[email protected]>

Co-authored-by: Svarshik <[email protected]>

…/jumpsuits.yml Co-authored-by: Svarshik <[email protected]>

Tris0r · 2024-12-17T12:24:58Z

Почему не мёрж

lexaSvarshik · 2024-12-17T14:15:48Z

Почему не мёрж

something broke

Exception whilst loading prototypes from /Prototypes/SS220/Entities/Clothing/Uniforms/Jumpsuit/jumpsuits.yml:
(Line: 75, Col: 145, Idx: 2591) - (Line: 75, Col: 183, Idx: 2629): While parsing a block mapping, did not find expected key. Exception:

Tris0r · 2024-12-17T14:49:51Z

Почему не мёрж

something broke

Exception whilst loading prototypes from /Prototypes/SS220/Entities/Clothing/Uniforms/Jumpsuit/jumpsuits.yml:
(Line: 75, Col: 145, Idx: 2591) - (Line: 75, Col: 183, Idx: 2629): While parsing a block mapping, did not find expected key. Exception:

А чё молчите тогда

# Conflicts: # Resources/Locale/ru-RU/ss220/clothing/Jumpskirt/jumpskirt.ftl # Resources/Locale/ru-RU/ss220/clothing/Jumpsuit/jumpsuit.ftl # Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpskirt/jumpskirts.yml

…CHEBUREKI # Conflicts: # Resources/Locale/ru-RU/ss220/clothing/Jumpskirt/jumpskirt.ftl # Resources/Locale/ru-RU/ss220/clothing/Jumpsuit/jumpsuit.ftl # Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpskirt/jumpskirts.yml

Tris0r · 2024-12-18T12:28:39Z

Изи все ошибки исправил.

stalengd · 2024-12-22T11:08:10Z

Resources/Prototypes/Loadouts/loadout_groups.yml

@@ -1378,6 +1378,8 @@
  - ChiefMedicalOfficerJumpskirt
  - ChiefMedicalOfficerTurtleneckJumpsuit
  - ChiefMedicalOfficerTurtleneckJumpskirt
+  - CMOCorpJumpsuit #ss220
+  - CMOCorpJumpskirt #ss220


Всё ещё не исправлено

stalengd · 2024-12-22T11:18:30Z

Resources/Prototypes/Loadouts/loadout_groups.yml

@@ -63,7 +63,7 @@
  - ClothingFootWrapsBlue
  - ClothingHandWrapsBlack
  - ClothingFootWrapsBlack
-  # SS220-Hand/FootWraps-More-Color-End
+  # -Hand/FootWraps-More-Color-End


stalengd · 2024-12-22T11:18:34Z

Resources/Prototypes/Loadouts/loadout_groups.yml

@@ -74,7 +74,7 @@
  - Glasses
  - GlassesJamjar
  - GlassesJensen
-  - GlassesCyber #SS220-Two Floppa Iteration
+  - GlassesCyber #-Two Floppa Iteration


stalengd · 2024-12-22T11:18:38Z

Resources/Prototypes/Loadouts/loadout_groups.yml

@@ -118,14 +118,14 @@
  loadouts:
  - CaptainHead
  - CaptainCap
-  #SS220-CapHeadLoadoutsChanges-Begin
+  #-CapHeadLoadoutsChanges-Begin


stalengd · 2024-12-22T11:18:44Z

Resources/Prototypes/Loadouts/loadout_groups.yml

+  #-CapHeadLoadoutsChanges-End
+  #-loadout-captain-begin


stalengd · 2024-12-22T11:18:47Z

Resources/Prototypes/Loadouts/loadout_groups.yml

  - CaptainWhiteHead
  - CaptainWhiteBeret
-  #ss220-loadout-captain-end
+  #-loadout-captain-end


where is my CHEBUREKI??

7dd482c

Tris0r changed the title ~~where is my CHEBUREKI??~~ Корпоративная одежда для глав.врача Dec 12, 2024

stalengd requested changes Dec 12, 2024

View reviewed changes

stalengd self-assigned this Dec 12, 2024

stalengd requested a review from EstKemran December 12, 2024 16:20

github-actions bot added the Status: Needs Review label Dec 12, 2024

EstKemran reviewed Dec 12, 2024

View reviewed changes

GdeMoyCheburek?

60adcf6

lexaSvarshik suggested changes Dec 12, 2024

View reviewed changes

Tris0r requested a review from stalengd December 13, 2024 08:24

Tris0r and others added 3 commits December 14, 2024 15:15

Update Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpskir…

912e222

…t/jumpskirts.yml Co-authored-by: Svarshik <[email protected]>

Update Resources/Prototypes/Loadouts/loadout_groups.yml

2b4c8dc

Co-authored-by: Svarshik <[email protected]>

Update Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpsuit…

7902d19

…/jumpsuits.yml Co-authored-by: Svarshik <[email protected]>

Tris0r added 4 commits December 18, 2024 09:26

Merge branch 'master' into CMO-NEW-CHEBUREKI

8726db6

Merge branch 'master' into CMO-NEW-CHEBUREKI

8f3218e

# Conflicts: # Resources/Locale/ru-RU/ss220/clothing/Jumpskirt/jumpskirt.ftl # Resources/Locale/ru-RU/ss220/clothing/Jumpsuit/jumpsuit.ftl # Resources/Prototypes/SS220/Entities/Clothing/Uniforms/Jumpskirt/jumpskirts.yml

Hokage

1436ae8

EstKemran approved these changes Dec 21, 2024

View reviewed changes

stalengd requested changes Dec 22, 2024

View reviewed changes

I want shaurma

9191671

stalengd requested changes Dec 22, 2024

View reviewed changes

stalen its not me

65b957f

stalengd approved these changes Dec 22, 2024

View reviewed changes

stalengd merged commit 0dd3f5f into SerbiaStrong-220:master Dec 22, 2024
15 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Корпоративная одежда для глав.врача #2345

Корпоративная одежда для глав.врача #2345

Tris0r commented Dec 12, 2024

github-actions bot commented Dec 12, 2024 •

edited

Loading

Tris0r commented Dec 12, 2024

stalengd Dec 12, 2024

stalengd Dec 12, 2024

Tris0r Dec 12, 2024

stalengd Dec 12, 2024

stalengd Dec 12, 2024

stalengd Dec 12, 2024

Tris0r Dec 12, 2024

lexaSvarshik Dec 12, 2024

stalengd Dec 12, 2024

stalengd Dec 12, 2024

Tris0r commented Dec 12, 2024

EstKemran Dec 12, 2024

Tris0r Dec 12, 2024

EstKemran Dec 12, 2024 •

edited

Loading

lexaSvarshik left a comment

lexaSvarshik Dec 12, 2024

stalengd Dec 12, 2024

lexaSvarshik Dec 12, 2024

stalengd Dec 18, 2024

stalengd Dec 22, 2024

Tris0r commented Dec 17, 2024

lexaSvarshik commented Dec 17, 2024

Tris0r commented Dec 17, 2024

Tris0r commented Dec 18, 2024

stalengd Dec 22, 2024

stalengd Dec 22, 2024

stalengd Dec 22, 2024

stalengd Dec 22, 2024

stalengd Dec 22, 2024

stalengd Dec 22, 2024

		- id: ClothingUniformJumpsuitCMOCorporate #SS220-CMOCorporate
		- id: ClothingUniformJumpskirtCMOCorporate #SS220 - CMOCorporate

	- CMOCorpJumpskirt #ss220
	- CMOCorpJumpsuit #SS220-CMOCorporate

Корпоративная одежда для глав.врача #2345

Корпоративная одежда для глав.врача #2345

Conversation

Tris0r commented Dec 12, 2024

Описание PR

github-actions bot commented Dec 12, 2024 • edited Loading

Tris0r commented Dec 12, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Tris0r commented Dec 12, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

EstKemran Dec 12, 2024 • edited Loading

Choose a reason for hiding this comment

lexaSvarshik left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Tris0r commented Dec 17, 2024

lexaSvarshik commented Dec 17, 2024

Tris0r commented Dec 17, 2024

Tris0r commented Dec 18, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Dec 12, 2024 •

edited

Loading

EstKemran Dec 12, 2024 •

edited

Loading