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

Niteman tweaks (mostly toward Nightmatrons) #790

Merged
merged 5 commits into from
Oct 1, 2024

Conversation

OctusGit
Copy link
Contributor

@OctusGit OctusGit commented Sep 25, 2024

About The Pull Request

  • Moves nightman/matrons to nobles, similar to the merchant. Gives them their own flag that doesn't share Jester (Lmao)
  • The nightman leather jacket now works for females, tweaked restrictions on some races that it doesn't look well on.
  • Removed nightman knife skill and their hunting knife. Changed with a coin pouch for holding their funds.

Why It's Good For The Game

Giving the nightman his own flag and actual department flag is just consistent coding and fixing overall incorrect code. Leather jacket is mostly to have a unique clothing item that isn't exactly like the queen's dress. Lastly knife skill is useless and the coin pouch is much more appreciated. They don't really need knife skills when they have level 4 wrestling and unarmed. Their whole focus is hand to hand, knife is garbo.

dreamseeker_IHgUJELo24

Pre-Merge Checklist

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

Copy link
Contributor

@Optimism333 Optimism333 left a comment

Choose a reason for hiding this comment

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

Certainly would NOT say knife is garbo, but if you think that a coin pouch would leave them better off, that's quite fine.

Any reason you took aasimar off the allowed race list for the jacket? They can be nitemen/nitematrons as well.

@ThePainkiller
Copy link
Contributor

Know what, you could instead swap knife skill and knife for a cudgel and maces 2. It'd help him more than a pouch of coins tbh.

@NPC1314 NPC1314 added the Merge conflict This PR won't compile due to merge conflics label Sep 28, 2024
Copy link
Contributor

@Optimism333 Optimism333 left a comment

Choose a reason for hiding this comment

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

Would still like an answer as to why aasimar have been removed from the allowed races for the jacket, given that they can also play as nitemen/nitematrons.

@OctusGit
Copy link
Contributor Author

Would still like an answer as to why aasimar have been removed from the allowed races for the jacket, given that they can also play as nitemen/nitematrons.

Oh, mostly was worried female sprites wouldnt work on them anymore. However im sure it will be fine and use the male sprites since their the same bodytype (I think???)

@OctusGit
Copy link
Contributor Author

Know what, you could instead swap knife skill and knife for a cudgel and maces 2. It'd help him more than a pouch of coins tbh.

Id prefer they didnt have a cudgel and just had the coin pouch instead. If they really want protection, they have their unarmed abilities, or they can hire help.

@ThePainkiller ThePainkiller added the Icons This PR adds, edits or deletes icons label Sep 30, 2024
Copy link
Contributor

@Optimism333 Optimism333 left a comment

Choose a reason for hiding this comment

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

Don't really have any remaining objections to this. A rare instance when we're removing a weapon skill on a role and giving them a utility item instead. And they do have pretty good unarmed, so we can see how it pans out.

Jacket looks decent as well.

@ThePainkiller ThePainkiller merged commit c18c919 into Darkrp-community:main Oct 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Icons This PR adds, edits or deletes icons Merge conflict This PR won't compile due to merge conflics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants