Skip to content
This repository has been archived by the owner on Feb 14, 2021. It is now read-only.

Fix minigame and clues, add Boss killcounts #4

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Fix minigame and clues, add Boss killcounts #4

wants to merge 7 commits into from

Conversation

rebecca-davies
Copy link

No description provided.

Mikan Mai and others added 3 commits September 21, 2020 01:16
Co-authored-by: Omar Assadi <omar@assadi.co.il>
Co-authored-by: Mikan <mikan.monkey92@gmail.com>
Co-authored-by: Omar Assadi <omar@assadi.co.il>
Co-authored-by: Mikan <mikan.monkey92@gmail.com>
Co-authored-by: Omar Assadi <omar@assadi.co.il>
Co-authored-by: Mikan <mikan.monkey92@gmail.com>
@OmarAssadi
Copy link

Resolves #2 - might be worth replacing the text-based bosses with emoji icons, like runelite or w/e

model/hiscore.go Outdated
Clueselite = 32
Cluesmaster = 33
Offset = Bhhunter
Overall = 0
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Overall = 0
Overall = iota
Attack
Strength
...

lets get rid of this awfulness and use this instead

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will take care of it tomorrow when I'm not hammered 👍

@notmeta
Copy link
Owner

notmeta commented Sep 20, 2020

Perhaps add asdf as the embed Thumbnail to make it look a little less bleak

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants