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

Return of the tailor #1246

Merged
merged 4 commits into from
Dec 16, 2024
Merged

Conversation

NPC1314
Copy link
Contributor

@NPC1314 NPC1314 commented Dec 15, 2024

About The Pull Request

Lots of roles are half-assed, like towner weaver for example. Identical to pilgrim one, lame, so, spruced up some. Also shaping up the roles so the shunned races don´t fill up the town and be lame, if you want to be shunned tailor just roll pilgrim, thats literally what they originally were meant for, outsiders thats a bit exotic maybe.

Tailor/Seamstress got new drip, key to their shop, start with bit more resources and luxury dye kit.
split off from the pilgrim role thats called weaver that doesnt get the house for obvious reasons.

Dye bins can be upgraded to expensive dye ones with the set, old craft for them scrapped.
Fancy hats with feather now require a feather to make, duh

Rogue basically get the dagger and cudgel switched back so dagger in belt and dagger skill primary, cudgel in backpack and cudgel skill secondary. Lock pick skill set to 3 instead of 4 for now,.

Stevedore now trained by monks of ravox, leans towards having great stats but shitty combat skills, except unarmed.
Cudgel dudes were ok concept before but think this might be more fun, sort of towner monks only not quite.

Smithy apprentice got their new pad sorted.

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

@NPC1314 NPC1314 added the Scheduled for testmerge This PR should be testmerged label Dec 16, 2024
@NPC1314 NPC1314 merged commit a162a30 into Darkrp-community:main Dec 16, 2024
7 checks passed
@NPC1314 NPC1314 removed the Scheduled for testmerge This PR should be testmerged label Dec 16, 2024
@NPC1314 NPC1314 deleted the weaverandstevedorerogue branch December 16, 2024 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant