Fix slot number for Perkpocalypse perks in calendar

Skytils / SkytilsMod

Skytils is a Hypixel Skyblock mod! Be careful, malicious copies are distributed across GitHub. Confirm on discord.gg/skytils (807302538558308352)

https://hypixel.net/threads/3856202/

GNU Affero General Public License v3.0

1.16k stars 442 forks source link

Fix slot number for Perkpocalypse perks in calendar #520

Closed lunaynx closed 1 month ago

lunaynx commented 1 month ago

Fixed Perkpocalypse perks not getting detected from the calendar due to the slot number being wrong.

Screenshot of calendar, showing mayor in slot 37

My-Name-Is-Jeff commented 1 month ago

Is the calendar permanently like this now? Or is there a case where the row would be added/removed.

lunaynx commented 1 month ago

I can't think of a case where it would be different, I remember the existing slot number was also wrong last Jerry when I made a PR to fix the condition being a || which ended up inadvertently breaking the check that happened to work before. Can't say for sure it was the exact same slot number then, but I think the calendar looked the same.