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

Guidebook Update #2090

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

Surani1
Copy link

@Surani1 Surani1 commented Oct 14, 2024

Описание PR

Рефактор гайдбуков, пока PR заброшен, как только руки дойдут пойду делать

  • перенос КЗ

Медиа
by UrPrice:
https://cdn.discordapp.com/attachments/1316067954554703902/1316067963627110461/NVIDIA_Overlay_yhNhLWNV5c.gif?ex=6759b34d&is=675861cd&hm=3e9cf469941e8c121ab0aced77e0dbb744de301cdd735084b9e66ec25778f83f&

Проверки

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

Изменения
🆑 Surani, UrPrice

  • tweak: Изменён гайдбук КЗ в игре.

@Surani1 Surani1 changed the title [WIP] [WIP] guidebook update Oct 14, 2024
@github-actions github-actions bot added the Changes: Sprites Изменение затронуло файлы ".rsu / .png" label Oct 14, 2024
Copy link

github-actions bot commented Oct 14, 2024

RSI Diff Bot; head commit 12c7c10 merging into b97188b
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Objects/Misc/books.rsi

State Old New Status
space_law Added

Edit: diff updated after 12c7c10

@Surani1
Copy link
Author

Surani1 commented Oct 16, 2024

А почему китовраса ревью?

Copy link

@Kit0vras Kit0vras left a comment

Choose a reason for hiding this comment

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

Просмотрел только часть, пока было время

Resources/ServerInfo/SS220/Guidebook/Security/Defusal.xml Outdated Show resolved Hide resolved
@Surani1
Copy link
Author

Surani1 commented Oct 16, 2024

Просмотрел только часть, пока было время

Я хуй знает зачем прайс запросил ревью, ибо это паста старых гайдбуков, а не создание новых. Я ничего еще пока не сделал, не считая переноса и разбора работы XML. Драфт висит же епта

@UrPrice
Copy link
Collaborator

UrPrice commented Oct 17, 2024

Просмотрел только часть, пока было время

Я хуй знает зачем прайс запросил ревью, ибо это паста старых гайдбуков, а не создание новых. Я ничего еще пока не сделал, не считая переноса и разбора работы XML. Драфт висит же епта

Из-за большого количества текстов, запросил его как редактора на логические, синтаксические, орфографические ошибки конкретно в тексте, чтобы потом поправлять всё это, несмотря на то, что это копипаста

@lexaSvarshik
Copy link

lexaSvarshik commented Oct 23, 2024

space-wizards#32894
Оффы переделывают инженерный гайдбук

space-wizards#32824
Минорные антаги

@ArtisticRoomba
Copy link

Hey y'all, I saw that you referenced my PR for your fork's guidebook update. Just wanted to make it known that I fucked up Git and reopened my PR, space-wizards#33062.

It'll probably be a while before it's finished but just know I'm still working on it.

Cheers :)

@UrPrice
Copy link
Collaborator

UrPrice commented Nov 2, 2024

Hey y'all, I saw that you referenced my PR for your fork's guidebook update. Just wanted to make it known that I fucked up Git and reopened my PR, space-wizards#33062.

It'll probably be a while before it's finished but just know I'm still working on it.

Cheers :)

Ty for info, mate
We'll keep this changes in mind

@UrPrice
Copy link
Collaborator

UrPrice commented Nov 5, 2024

@Surani1 А чего тут осталось доделать? Помочь мб?

@Surani1
Copy link
Author

Surani1 commented Nov 20, 2024

@Surani1 А чего тут осталось доделать? Помочь мб?

Уже не помню, я забил и продвижения не будет. Можете посмотреть, да доделать

@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Dec 10, 2024
@UrPrice UrPrice marked this pull request as ready for review December 10, 2024 15:42
@UrPrice UrPrice changed the title [WIP] guidebook update Guidebook Update Dec 10, 2024
@UrPrice
Copy link
Collaborator

UrPrice commented Dec 10, 2024

Доделал гайдбук, добавил гиф в медиа. Вроде из КЗ это всё.

@UrPrice
Copy link
Collaborator

UrPrice commented Dec 10, 2024

@Kit0vras посмотришь на орфографию? Я хоть и из Вики копировал, но всё равно.

@UrPrice UrPrice requested review from Kit0vras and stalengd December 10, 2024 15:45
@stalengd
Copy link
Collaborator

@Kit0vras посмотришь на орфографию? Я хоть и из Вики копировал, но всё равно.

Можешь spellcheck на vs code поставить

@stalengd
Copy link
Collaborator

@UrPrice и да, тебе возможно стоит прописаться в ченджлог раз ты доделывал

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Dec 16, 2024
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png" Merge Conflict Some content has conflicts. Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants