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

Catacombs Completion Pushing NaN #18

Open
WidthDrew opened this issue Dec 4, 2023 · 3 comments
Open

Catacombs Completion Pushing NaN #18

WidthDrew opened this issue Dec 4, 2023 · 3 comments

Comments

@WidthDrew
Copy link

WidthDrew commented Dec 4, 2023

I keep throwing an error where I am getting NaN for catacombs completions. For both base and master.

@Antonio32A @Desco1 @LappySheep @nstringham

@Antonio32A
Copy link
Owner

I believe the Hypixel API has changed over the years so some field is probably missing (I haven't played SB in years) I'll take a look over the weekend and see what I can do.

@WidthDrew
Copy link
Author

I believe the Hypixel API has changed over the years so some field is probably missing (I haven't played SB in years) I'll take a look over the weekend and see what I can do.

I believe by looking at the API - theres a new line under tier_completions for dungeons stating the total number of runs for all floors. I think that may be causing the issue.

@LappySheep
Copy link
Collaborator

I can have a look at it later if I'm free. The code my Discord bot runs seems to do fine.

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

No branches or pull requests

3 participants