Skip to content

Master Tier work + misc changes - #101

Open
Tesla3103 wants to merge 3 commits into
OSRS-Taskman:mainfrom
Tesla3103:master-tier-and-misc
Open

Tesla3103 wants to merge 3 commits into
OSRS-Taskman:mainfrom
Tesla3103:master-tier-and-misc

Conversation

@Tesla3103

Copy link
Copy Markdown
Contributor

Master tier rework

  • Rework the master, extra and passive tiers to match the approved master task list.
  • Add verification methods for every task, with the exception of Combat Achievement tasks.
  • Merge the individual boss pet and jar tasks into a unified format.
    • This means that the pet.json file is now redundant. It is however left untouched for legacy purposes.
  • Add a category tag to every master tier task.
    • Consequently, add the appropriate category tags to the task schema.

Chompy Hat rework

  • Rework and unify the chompy bird hat tasks. Specifically:
  • Easy tier:
    • Rename 1x "Get the Marksman headgear" to "Get the next 2 tiers of Chompy bird hat"
    • Add 2x "Get the next 2 tiers of Chompy bird hat"
  • Medium tier:
    • Rename 1x "Get the Ogre forester headgear" to "Get the next 2 tiers of Chompy bird hat"
    • Add 1x "Get the next 2 tiers of Chompy bird hat"
  • Hard tier:
    • Rename 1x "Get the Ogre expert headgear" to "Get the next 2 tiers of Chompy bird hat"
    • Add 1x "Get the next 2 tiers of Chompy bird hat"
  • Elite tier:
    • Rename 1x "Upgrade to the dragon archer headpiece" to "Get the next 2 tiers of Chompy bird hat"
    • Rename 1x "Upgrade to the expert dragon archer headpiece" to "Get the next 2 tiers of Chompy bird hat"

Misc changes

  • Remove "(Cox or ToB or ToA)" from every Raids task name
  • Add The Fractured Archive in every Raids task tooltip in preparation for the upcoming raid.

Comment thread schema/task.schema.json
"tag": {
"type": "string",
"enum": ["lms"]
"enum": ["lms", "Boss", "Raid", "Clue", "Misc", "BPet", "SPet", "CA", "L99"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do these need to be shortened? Can't they just be "Boss Pet" / "Skilling Pet" / "Combat Achievement" / "Level 99"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

boss-pet, skilling-pet, etc. pls

@rmobis rmobis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Still gotta check the master and extra tier, but other than the few comments I left, all looking good.

Comment thread tiers/easy.json
}
},
{
"id": "ecf1d31c-086c-4a29-9875-1f3cacabd11b",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This task (same count) keeps the old id. Previous ones get new ids.

Comment thread tiers/elite.json
"name": "Get 1 unique from Raids",
"shortName": "Raids",
"tip": "Can be received as a drop from any raid: Chambers of Xeric, Theatre of Blood or Tombs of Amascut.",
"tip": "Can be received as a drop from any raid: Chambers of Xeric, Theatre of Blood, Tombs of Amascut or The Fractured Archive.",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Feel like explicitly naming FRED before it can actually complete the task will just get people asking if they can do their raids task there (which I believe we decided they couldn't?)

Comment thread tiers/passive.json
"imageLink": "https://oldschool.runescape.wiki/images/thumb/Mystic_boots_(dusk)_detail.png/320px-Mystic_boots_(dusk)_detail.png",
"displayItemId": 23059
"wikiLink": "https://oldschool.runescape.wiki/w/Brimstone_chest",
"imageLink": "https://oldschool.runescape.wiki/images/thumb/Brimstone_key_detail.png/1024px-Brimstone_key_detail.png",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

320px please?

Comment thread tiers/passive.json
"name": "Get a Gilded full helm",
"tip": "Obtained very rarely from hard, elite and master clues.",
"wikiLink": "https://oldschool.runescape.wiki/w/Gilded_full_helm",
"imageLink": "https://oldschool.runescape.wiki/images/thumb/Gilded_full_helm_detail.png/110px-Gilded_full_helm_detail.png",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

320px pls?

This branch has not been deployed

No deployments
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