Skip to content

feat: Added new pet upgrade recipe framework#10

Open
Schemb wants to merge 1 commit intoNotEnoughUpdates:masterfrom
Schemb:direct_pet_upgrade
Open

feat: Added new pet upgrade recipe framework#10
Schemb wants to merge 1 commit intoNotEnoughUpdates:masterfrom
Schemb:direct_pet_upgrade

Conversation

@Schemb
Copy link
Copy Markdown

@Schemb Schemb commented May 1, 2026

A new recipe framework for pets that are upgraded with an upgrade stone rather than at Kat. Such as the 'Griffin' and 'Wisp'

…ed with an 'upgrade stone'; such as the 'Griffin' and 'Wisp'
@lineargraph
Copy link
Copy Markdown
Member

lineargraph commented May 1, 2026

is this in conjunction with a repo PR or already actively present recipe type in the repo? if not and you want it to be, i feel like this should be discussed in the neu repo itself, or the corresponding discord channels. to be clear: this git repository is not the place to discuss design decisions about the json structure of the data repo.

@lunaynx
Copy link
Copy Markdown

lunaynx commented May 1, 2026

For what it's worth, Hypixel has been moving away from this system. Griffin was changed to use Kat with Diana v2, and I think Wisp is the only one still using it. But nobody can predict how long it will take them to actually change Wisp, so it may be worth having some way to express that upgrade path in the repo.

@Schemb
Copy link
Copy Markdown
Author

Schemb commented May 1, 2026

It's for the issue in the NEU repo raised about the Griffin Pet having a missing recipe NotEnoughUpdates/NotEnoughUpdates-REPO#2417 which I presumed was missing because there wasn't a way to express that it's upgraded differently so I thought adding a new category first was necessary, sorry if I've skipped a better place to propose it beforehand.

@lunaynx Thank you, I'll look into that Griffin change too and see if I can get the Kat recipes added. They aren't listed on the wiki yet so I was unaware it'd been changed.

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.

3 participants