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

Sorting Departments. #42

Merged
merged 74 commits into from
Nov 18, 2024

Conversation

Adrian16199
Copy link
Contributor

@Adrian16199 Adrian16199 commented Jan 18, 2024

Description

Sorts the departments. Cargo being in service, salvagers being in engineering, HoP having low AA.


Changelog

🆑

  • tweak: Some jobs have been adjusted! Some are in new departments, some gained access.

TO-DO:
-Get rid of certain syndicate objectives
-Find out how to get rid of Logistics in department.yml
-Add service supervisor from parkstation
-probably adjust icons to what they were in parkstation
-get rid of shoukou when its time to actualy get this on review.
@github-actions github-actions bot added Changes: C# Changes any cs files Changes: Map Changes any yml file in the Maps directories Status: Merge Conflict FIX YOUR PR AAAGH labels Jan 18, 2024
Copy link
Contributor

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

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Jan 25, 2024
@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Jan 28, 2024
Copy link
Contributor

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

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Jan 30, 2024
@github-actions github-actions bot added the Changes: Sprite Changes any png or json in an rsi label Jan 30, 2024
Copy link
Contributor

github-actions bot commented Jan 30, 2024

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

Resources/Textures/DeltaV/Interface/Misc/job_icons.rsi

State Old New Status
nyanoMailCarrier Modified

Resources/Textures/Interface/Misc/job_icons.rsi

State Old New Status
CargoTechnician Modified

Resources/Textures/Mobs/Customization/reptilian_parts.rsi

State Old New Status
tail_smooth_wagging Removed

Edit: diff updated after ee715b5

@github-actions github-actions bot added Changes: Localization Changes any ftl files Changes: YML Changes any yml files and removed Changes: C# Changes any cs files Changes: Map Changes any yml file in the Maps directories labels Feb 1, 2024
@Adrian16199 Adrian16199 marked this pull request as ready for review February 1, 2024 13:37
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Feb 1, 2024
@Adrian16199
Copy link
Contributor Author

Pretty much ready for review, no idea why it failed the test debug so would be cool if i was told why

Copy link
Member

@DEATHB4DEFEAT DEATHB4DEFEAT left a comment

Choose a reason for hiding this comment

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

Is that really how DV does their codebase change markers? That sucks.

Resources/Locale/en-US/nyanotrasen/job/job-description.ftl Outdated Show resolved Hide resolved
Resources/Prototypes/DeltaV/Catalog/Fills/Boxes/pda.yml Outdated Show resolved Hide resolved
@DEATHB4DEFEAT DEATHB4DEFEAT added Size: 3-Medium For medium issues/PRs Status: Requires This requires something else to be done before being resolved Priority: 2-High Needs to be resolved as soon as possible labels Feb 1, 2024
Copy link
Member

@Pspritechologist Pspritechologist left a comment

Choose a reason for hiding this comment

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

You're certain that removing Captain from the Department list is as effective as labeling them not selectable? I'd still like to see it disabled, personally.

Resources/Prototypes/Entities/Clothing/Ears/headsets.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Entities/Clothing/Head/soft.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Entities/Clothing/Head/soft.yml Outdated Show resolved Hide resolved
primary: false
# Parkstation-Oligarchy Start
#- type: department
# id: Specific
Copy link
Member

Choose a reason for hiding this comment

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

What's this? DeltaV (or Wizden) delegates their less important roles to a weird fake Department? Lame as hell. Move Psych into Medical and Reporter into Service, please.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

uh, it uh, Itss already like that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At the least those 2 roles that you said.

Copy link
Member

Choose a reason for hiding this comment

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

It's possible I was looking at an old commit then? I believe the "department" wasn't commented out when I left that, but I may have had a commit filter on by mistake.

@Adrian16199
Copy link
Contributor Author

You're certain that removing Captain from the Department list is as effective as labeling them not selectable? I'd still like to see it disabled, personally.

Well.....
That would require to update the dev map via map and what roles are avaible on it.
This solution just simply bypasses that issue and you can still just hit observe, if you tell me that you still want to see that change, fair enough.

@@ -1,7 +1,7 @@
- type: entity
parent: BaseItem
id: SpaceCashLuckyBill
name: quartermaster's lucky bill
name: Cargo technician's lucky bill
description: The first bill ever earned by the Cargo Technician from trading. A small chip is embedded in it.
Copy link
Member

Choose a reason for hiding this comment

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

Cargo tech isn't a singular role, so 'the Cargo Tech' doesn't make sense.

Comment on lines 34 to 35
#- type: PaintableAirlock
# department: Cargo
Copy link
Member

Choose a reason for hiding this comment

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

Commenting these out probably isn't the best solution.

Comment on lines 216 to 217
#- type: PaintableAirlock
# department: Cargo
Copy link
Member

Choose a reason for hiding this comment

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

Ditto

@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Mar 16, 2024
Copy link
Contributor

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

@SimpleStation14 SimpleStation14 changed the title Sorting departments. Sorting Departments. Nov 17, 2024
@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Nov 17, 2024
@github-actions github-actions bot added Changes: Documentation Changes any xml or md files Changes: Config Changes any configuration file labels Nov 17, 2024
@DEATHB4DEFEAT DEATHB4DEFEAT merged commit 3bbf04a into Simple-Station:master Nov 18, 2024
13 checks passed
Pspritechologist pushed a commit that referenced this pull request Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Config Changes any configuration file Changes: Documentation Changes any xml or md files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files Priority: 2-High Needs to be resolved as soon as possible Size: 3-Medium For medium issues/PRs Status: Needs Review Someone please review this Status: Requires This requires something else to be done before being resolved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants