-
Notifications
You must be signed in to change notification settings - Fork 100
Changed the categories of NPCs to reflect their race and not their name. #1292
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
Closed
PrettyPrincessPotato
wants to merge
6
commits into
GregHib:main
from
PrettyPrincessPotato:main-slayer-categories-fix
Closed
Changes from all commits
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
97fcb73
init
f3c7c69
Added slayer xp and fixed typos in the categories. Properly tested co…
54c354d
Merge branch 'main' into main-slayer-categories-fix
PrettyPrincessPotato ffd310e
Merge branch 'main' into main-slayer-categories-fix
PrettyPrincessPotato 1a092fd
Update data/entity/npc/monster/undead/corpse/corpse.npcs.toml
GregHib 380fb3b
Merge branch 'main' into main-slayer-categories-fix
PrettyPrincessPotato File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't look like they gave slayer_xp in 2011, they were only added in the 2023 slayer update.
https://runescape.wiki/w/Corpse_spider?oldid=4633922
|slaylvl = NoThey do not count as spiders for Slayer tasks.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey that's handy, is
?oldid=4633922interchangeable on all wiki links? So far I've been just looking at the current wiki, any updates posted at the bottom, and guessing.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately not but you can find the relevant one by clicking the history button top right and going back to a date ~2011
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see then, it appears none of them were actually able to be used for slayer tasks.