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

update clothing vendor in squad prep to hold all clothing items and use scaling. #4065

Merged
merged 15 commits into from
Aug 11, 2023

Conversation

Huffie56
Copy link
Contributor

@Huffie56 Huffie56 commented Jul 31, 2023

About the pull request

The idea is to move all the clothing items to the uniform vendor.
then turn the Armaments vendor into an utilities vendor and populate it with usefull item that you can find around the ship for free...
I am also adding welding google and high capacity blowtorch are people can loot them on the ship and their is plenty of them
no reason to make people loot maintenance or department to complete their favorite loadout if those are items are widely available....

Explain why it's good for the game

It regroup all the clothing items in a single vendor to avoid have multiple vendors selling the same item and refocus the utilities vendor toward another set of items...

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
code: added welding google to the com tech tool vendor.
code: added scaling based on round population to the ColMarTech Surplus Uniform like the other vendors.
code: Move every clothing items from ColMarTech Automated Armaments Squad Vendor to ColMarTech Surplus Uniform Vendor.
code: rename ColMarTech Automated Armaments Squad Vendor to ColMarTech Automated Utilities Squad Vendor and add in tools food and other utilities.
/:cl:

to use scaling and  move all the clothings
items of the other vendors in squad preps to
it.
the idea is to have vendors with a designated purpose
and category of items they hold.
@github-actions github-actions bot added the Code Improvement Make the code longer label Jul 31, 2023
@Huffie56
Copy link
Contributor Author

tag me on discord if you have suggestion...

Julien added 2 commits July 31, 2023 16:24
and empty medikit to vendors.
add three glasses to the uniform vendor.
@Huffie56 Huffie56 marked this pull request as ready for review August 1, 2023 11:30
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Aug 2, 2023
@cm13-github
Copy link
Contributor

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

@Huffie56 Huffie56 marked this pull request as draft August 2, 2023 07:31
@Huffie56 Huffie56 marked this pull request as ready for review August 2, 2023 08:00
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Aug 2, 2023
@Huffie56 Huffie56 marked this pull request as draft August 2, 2023 08:04
missing an indentation
@Huffie56 Huffie56 marked this pull request as ready for review August 2, 2023 09:42
Huffie56 and others added 2 commits August 2, 2023 11:43
add an empty to force rerun of test
Copy link
Member

@morrowwolf morrowwolf left a comment

Choose a reason for hiding this comment

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

Do another pass on scale changes for restricted gear (although god knows how you would do this accidentally) and then let me know.

@morrowwolf morrowwolf marked this pull request as draft August 6, 2023 11:48
@Huffie56 Huffie56 marked this pull request as ready for review August 6, 2023 12:05
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Aug 8, 2023
@cm13-github
Copy link
Contributor

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

@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Aug 9, 2023
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Aug 9, 2023
@cm13-github
Copy link
Contributor

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

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Aug 9, 2023
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@morrowwolf morrowwolf added this pull request to the merge queue Aug 11, 2023
@Huffie56
Copy link
Contributor Author

thanks this keep getting merge conflicts....

Merged via the queue into cmss13-devs:master with commit 23ab6ba Aug 11, 2023
25 checks passed
cm13-github added a commit that referenced this pull request Aug 11, 2023
@Huffie56 Huffie56 deleted the clear_vendor_roles branch August 15, 2023 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Make the code longer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants