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 cube recipes for gems (and renamed recipes for upgrading) #535

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

IronMaXxX
Copy link

I added the missing recipes for upgrading all gems from chipped to perfect gems.
Renamed the recipes for gems to match the terminology of runes (Upgrading ...), so far the naming has been contradictory.

Crafting recipes are available within cubing_recipes.go but were missing in recipes.go.
Not sure if this was/is intentional, added them for the sake of completeness.

=> is it intended to hide those recipes? If no the comment // could be deleted to activate the recipes.
@maintainers: feel free to edit the file if needed

Added Cube recipes for upgrading gems from chipped to perfect, renamed gem recipes to match the rune recipes (upgrade ...)
Renamed existing recipes, added recipes for gems to match the recipes for runes (Upgrade ...)

Crafting recipes are available within cubing_recipes.go but were missing here.  Not sure if this was/is intentional, added them for the sake of completeness.

=> is it intended to hide those recipes?
@artosimonyan
Copy link
Collaborator

Still not sure about this one. It will make the settings UI hideous with all those recipes.

Also, not sure about the naming convention yet (if it should be mixed, same or w/e). Have to think about it

@davidfvsilva
Copy link
Contributor

Still not sure about this one. It will make the settings UI hideous with all those recipes.

Also, not sure about the naming convention yet (if it should be mixed, same or w/e). Have to think about it

Could do with some cleanup, but having those recipes is a step forward. As for the UI concern, we could just rework the UI with some collapsing menus or something, but I think @hectorgimenez is already working on something.

@davidfvsilva
Copy link
Contributor

I've added this to my dev branch and there is no issue. Blood gauntlets get crafted.

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