Speaax / Farming-Helper

Helps with farming
BSD 2-Clause "Simplified" License
0 stars 4 forks source link

Bad item requirements using Skills necklace #13

Open Speaax opened 8 months ago

Speaax commented 8 months ago

By selecting 2 or more teleport options using Skills necklace you will be required to take out 2 or mote Skill necklaces during the item requirements phase.

This needs to be updated to count how many teleport charges are left on Skills necklace in inventory, and only tell you to bring another one if the teleport charges are less than the teleport charges needed for a full run.

othyn commented 8 months ago

Will need the solution to this to also apply to the Digsite pendant for Fossil Island too, once I've finished #12

Speaax commented 8 months ago

Probably a good idea to implement it there as well, but I don't think there is a scenario where the item requirements will add a digsite pendant twice as you only need to go there once.

But yeah, I guess it should be implemented for all items that has charges anyways, so there is consistency.

othyn commented 8 months ago

In relation to #12, I ended up having to add the hardwood tree patches as 3 separate Fossil Island patches due to the 1:1 linking of location and farming patch. So it will technically initially require 3 digsite charges, when in reality you only need and use one.