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

Rework races guidebook #2060

Merged
merged 7 commits into from
May 10, 2024
Merged

Rework races guidebook #2060

merged 7 commits into from
May 10, 2024

Conversation

fqqf
Copy link
Contributor

@fqqf fqqf commented Apr 24, 2024 •
edited
Loading

Описание PR

Адаптация руководств по расам на русский язык. Добавлен раздел для расы вульпканинов. Некоторые описания рас переписаны в соответствии с лором и добавлено описание расовых особенностей для каждой расы.

Медиа

out44.mp4

Проверки

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

Изменения

🆑 Miracula

  • tweak: Руководство по расам адаптировано под лор и переведено на русский язык, добавлены новые разделы.

@fqqf fqqf requested a review from Morb0 as a code owner April 24, 2024 17:30
@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged No C# Code has not been changed labels Apr 24, 2024
@lzk228
Copy link
Collaborator

lzk228 commented Apr 24, 2024

Не одобряю
За разными добавлениями, добро пожаловать к https://github.com/space-wizards/space-station-14
у нас перевод, добавлений нету

@fqqf
Copy link
Contributor Author

fqqf commented Apr 24, 2024 •
edited
Loading

Не одобряю За разными добавлениями, добро пожаловать к https://github.com/space-wizards/space-station-14 у нас перевод, добавлений нету

У оффов нет расы вульп, тут без добавлений не выйдет. Это не перевод, а скорее реворк, так как в изначальном руководстве вообще почти ничего не сказано о расовых особенностях. Плюс у корвакса свой лор, и сделать ПР об этом лоре у оффов, ну, банально не выйдет, понятно почему

@lzk228
Copy link
Collaborator

lzk228 commented Apr 24, 2024

я говорю не только о вульпах
все реворки и прочее сюда https://github.com/space-wizards/space-station-14
у нас только перевод, естественно с добавлением гайда о вульпах

@fqqf
Copy link
Contributor Author

fqqf commented Apr 24, 2024

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

В любом случае, напишите о своем решении, как рассмотрите. Можно ли вернуть полную версию руководства, или все же разрешаете оставить только часть про вульп

@themanyfaceddemon
Copy link

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

В любом случае, напишите о своем решении, как рассмотрите. Можно ли вернуть полную версию руководства, или все же разрешаете оставить только часть про вульп

Для того чтобы был отдел лороведов и тонна никому не нужного текста

@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label Apr 27, 2024
@github-actions github-actions bot removed the Merge Conflict PR have unsolved merge conflicts label Apr 27, 2024
@fqqf fqqf requested a review from Morb0 April 27, 2024 15:56
@fqqf
Copy link
Contributor Author

fqqf commented Apr 28, 2024

Собственно, вроде все)

lzk228
lzk228 requested changes Apr 28, 2024
View reviewed changes
Resources/Prototypes/Guidebook/species.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Guidebook/species.yml Outdated
Comment on lines 50 to 53
- type: guideEntry
id: Vulpkanin
name: species-name-vulpkanin
text: "/ServerInfo/Guidebook/Mobs/Vulpkanin.xml"
Copy link
Collaborator

Choose a reason for hiding this comment

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

этот отдельный прототип можно вывести в папку корвакс
`Resources/Prototypes/Corvax/Guidebook/species.yml

Resources/ServerInfo/Guidebook/Mobs/Vulpkanin.xml Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

на счёт вынесения этого файла в папку корвакс спорно, пусьт морти уточнит

@Morb0 Morb0 merged commit 7c46a9f into space-syndicate:master May 10, 2024
9 checks passed
@fqqf fqqf deleted the rework-races-guidebook branch May 22, 2024 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lapatison lapatison lapatison left review comments

@lzk228 lzk228 lzk228 requested changes

@Morb0 Morb0 Awaiting requested review from Morb0 Morb0 is a code owner

Assignees
No one assigned
Labels
No C# Code has not been changed Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@fqqf @lzk228 @themanyfaceddemon @Morb0 @lapatison