WIP: Seafluff Guidebook changes. #1043

Draft
Allycat wants to merge 9 commits from Allycat/ByrdAllycat:GuidebookAllycat into master
First-time contributor

Short description

Adding Byrd section, with SOP and space law and other updates.

Why we need to add this

Seafluff's Guidebook changes, I just did coding and fixed some errors. Seafluff can answer queries about need.

Checklist

Breaking changes

None

<!-- Link definitions, do not edit --> [guidelines]: https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html [ruleset]: https://rules.byrdstation.space/#h.vigrxrnffwji ## Short description Adding Byrd section, with SOP and space law and other updates. ## Why we need to add this Seafluff's Guidebook changes, I just did coding and fixed some errors. Seafluff can answer queries about need. ## Checklist <!-- Check boxes for faster reviewing of your PR --> - [X] I do not require assistance to complete the PR. - [X] I have verified that the changes work before posting/requesting review of my PR. - [X] I have added screenshots/videos of the changes, or this PR does not change in-game mechanics. - [X] I have read the [Pull Request and Changelog Guidelines][guidelines] and [Byrd Station's Contributor Ruleset][ruleset]. ## Breaking changes None
Resources/Prototypes/_Byrd/Guidebook
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Failing after 4s
PR Changelogs / changelog (pull_request_target) Has been skipped
CRLF Check / CRLF Check (pull_request) Has been cancelled
Test Packaging / Test Packaging (pull_request) Has been cancelled
RGA schema validator / YAML RGA schema validator (pull_request) Has been cancelled
RSI Validator / Validate RSIs (pull_request) Has been cancelled
Map file schema validator / YAML map schema validator (pull_request) Has been cancelled
YAML Linter / YAML Linter (pull_request) Has been cancelled
2b5b114ede
Resources/Locale/en-US/guidebook
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Failing after 3s
CRLF Check / CRLF Check (pull_request) Failing after 35s
Test Packaging / Test Packaging (pull_request) Successful in 15m28s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 45s
RSI Validator / Validate RSIs (pull_request) Successful in 44s
Map file schema validator / YAML map schema validator (pull_request) Successful in 5m44s
YAML Linter / YAML Linter (pull_request) Successful in 5m34s
7e2d32f4f3
EchoOfNothing left a comment

The SOP is not accessible in-game.
guidebook
You have to place xml and yaml files under the Resources directory/folder.
In this case, yaml files in Resources/Prototypes/_Byrd/Guidebook and xml files in Resources/ServerInfo/Guidebook/_Byrd (Additional folder can be made to make it look nicer, more organized)

The SOP is not accessible in-game. ![guidebook](/attachments/96eed5df-58aa-4394-9267-943f06e56f02) You have to place xml and yaml files under the `Resources` directory/folder. In this case, yaml files in `Resources/Prototypes/_Byrd/Guidebook` and xml files in `Resources/ServerInfo/Guidebook/_Byrd` (Additional folder can be made to make it look nicer, more organized)
Owner

I also recommend hiding the old sop to avoid confusion

I also recommend hiding the old sop to avoid confusion
Author
First-time contributor

@EchoOfNothing wrote in #1043 (comment):

The SOP is not accessible in-game. guidebook You have to place xml and yaml files under the Resources directory/folder. In this case, yaml files in Resources/Prototypes/_Byrd/Guidebook and xml files in Resources/ServerInfo/Guidebook/_Byrd (Additional folder can be made to make it look nicer, more organized)

Scroll down. It's in a different section labeled Byrd.

However I cannot say why, as I was organizing it based on Seafluff's direction. If changes need to be made, they should be involved in it.

@EchoOfNothing wrote in https://git.byrdstation.space/Byrd-Station/Byrd/pulls/1043#issuecomment-9083: > The SOP is not accessible in-game. [![guidebook](/attachments/96eed5df-58aa-4394-9267-943f06e56f02)](/Byrd-Station/Byrd/attachments/96eed5df-58aa-4394-9267-943f06e56f02) You have to place xml and yaml files under the `Resources` directory/folder. In this case, yaml files in `Resources/Prototypes/_Byrd/Guidebook` and xml files in `Resources/ServerInfo/Guidebook/_Byrd` (Additional folder can be made to make it look nicer, more organized) Scroll down. It's in a different section labeled Byrd. However I cannot say why, as I was organizing it based on Seafluff's direction. If changes need to be made, they should be involved in it.
Owner

@Allycat wrote in #1043 (comment):

Scroll down. It's in a different section labeled Byrd.

However I cannot say why, as I was organizing it based on Seafluff's direction. If changes need to be made, they should be involved in it.

There is no where left to scroll that's the screenshot of the entire guidebook. Byrd category doesn't exist because the files were not place where they belong. And also Seafluff told me that the Byrd category is supposed to replace the old sop category (if you don't know how to do this tell me, I will explain how to achieve this in more detail).

@Allycat wrote in https://git.byrdstation.space/Byrd-Station/Byrd/pulls/1043#issuecomment-9116: > Scroll down. It's in a different section labeled Byrd. > > However I cannot say why, as I was organizing it based on Seafluff's direction. If changes need to be made, they should be involved in it. There is no where left to scroll that's the screenshot of the entire guidebook. `Byrd` category doesn't exist because the files were not place where they belong. And also Seafluff told me that the `Byrd` category is supposed to replace the old sop category (if you don't know how to do this tell me, I will explain how to achieve this in more detail).
@ -0,0 +69,4 @@
name: guide-entry-byrdknowledge
priority: 6
text: "/ServerInfo/Guidebook/_Byrd/ByrdKnowledge.xml"
#ByrdStation SOP Entries begin here
Owner

I recommend replacing #ByrdStation SOP Entries begin here with #region SOP or #region ByrdStation SOP (but I think adding ByrdStation here is quite redundant since it's already in the file called Byrd.yml) and #Byrdstation SOP entries end with #endregion. This way most IDEs will let you collapse those regions nicely, making reading and navigating the file easier.

I recommend replacing `#ByrdStation SOP Entries begin here` with `#region SOP` or `#region ByrdStation SOP` (but I think adding `ByrdStation` here is quite redundant since it's already in the file called `Byrd.yml`) and `#Byrdstation SOP entries end` with `#endregion`. This way most IDEs will let you collapse those regions nicely, making reading and navigating the file easier.
Author
First-time contributor

I am aware of how to do it, it seems there just was A. a miscommunication with Seafluff and myself, and B. an error when the PR was made. I'm newer to the forgejo/codeberg setup and so likely mixed up a couple things when manually making the PR (I manually uploaded the files and likely swapped guidebook and _byrd in location).

But it was all well organized, for the purposes I was aware of.

If you'd like to discuss more, I'd rather we just talk on discord if that works

I am aware of how to do it, it seems there just was A. a miscommunication with Seafluff and myself, and B. an error when the PR was made. I'm newer to the forgejo/codeberg setup and so likely mixed up a couple things when manually making the PR (I manually uploaded the files and likely swapped guidebook and _byrd in location). But it was all well organized, for the purposes I was aware of. If you'd like to discuss more, I'd rather we just talk on discord if that works
QuillTheCreature changed title from Seafluff Guidebook changes. to WIP: Seafluff Guidebook changes. 2026-10-22 19:39:41 +00:00
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Failing after 3s
CRLF Check / CRLF Check (pull_request) Failing after 35s
Test Packaging / Test Packaging (pull_request) Successful in 15m28s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 45s
RSI Validator / Validate RSIs (pull_request) Successful in 44s
Map file schema validator / YAML map schema validator (pull_request) Successful in 5m44s
YAML Linter / YAML Linter (pull_request) Successful in 5m34s
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u "GuidebookAllycat:Allycat-GuidebookAllycat"
git switch "Allycat-GuidebookAllycat"
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Byrd-Station/Byrd!1043
No description provided.