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

Added training sword + Improved training sword #727

Merged
merged 5 commits into from
Dec 4, 2024

Conversation

6SpadesLuke
Copy link
Contributor

Description

Adds a longsword to the game

Crafted using these materials at a weapons workbench
2x Leather
2x Brass Ingots
5x Iron Ingots
3x Wooden Planks

Longsword stats :
25 Slash melee damage
20 Slash + 10 Pierce thrown

Also buffs the training swords a bit
5 Stamina damage ---> 12 Stamina damage
5 Thrown stamina damage ---> 8 Stamina damage

Media

The training sword :

LongSword

Adds a craftable longsword to the game
Copy link
Contributor

github-actions bot commented Nov 25, 2024

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

Resources/Textures/_Nuclear14/Objects/Weapons/Melee/long_sword.rsi

State Old New Status
icon Added
inhand-left Added
inhand-right Added

Edit: diff updated after 4d020a2

not sure if this'll fix it but I'm gonna check
@6SpadesLuke
Copy link
Contributor Author

this was not as easy as I thought

I believe this will fix the issue
@Peptide90
Copy link
Collaborator

Unable to load RSI '/Textures/_Nuclear14/Objects/Weapons/Melee/long_sword.rsi if you can figure out why or I can look.

@6SpadesLuke
Copy link
Contributor Author

I can look into it. no idea what the hell's causing it

Copy link
Collaborator

Choose a reason for hiding this comment

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

This is your issue. Your path is Resources/Textures/Objects whereas the one in code is _Nuclear14/Objects/Weapons/Melee/long_sword.rsi

It's now in N14 instead of normal
@TheDoktorZ TheDoktorZ merged commit 98b228e into Vault-Overseers:master Dec 4, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants