Skip to content

Quest resource tweaks

bushtail requested to merge bushtail/2009dev:quest-capi-clarity into master

What have you done in this MR?

Two ContentAPI functions were renamed:

  • getQP -> getQuestPoints
  • questStage -> getQuestStage

Two ContentAPI functions were added:

  • getQuest : Quest
  • sendItemZoomOnInterface

Changed quest journal text colours to be authentic. Quest Complete's red can now be called using %% &&.

!1365 (closed) (closed) has been merged with this MR with Ceikry's permission.

Are there any tricky things testers should keep an eye out for?

  • No
  • Yes, as follows:

Nothing tricky per-se, all you need to do is test something that checks quest points (for instance, trying to right click -> trade on the guy on the third floor of the Legends Guild with under 40 quest points), and something that checks quest stage (for instance, just do sheep shearer as normal)

  • I have tested these changes thoroughly.

  • This requires extra testing due to changes to architecture or other similarly risky changes.

  • I used the Thanos Tool for any JSON edits where possible, and have attached screenshots of any changes.

  • I acknowledge that this contribution will be released under the AGPL license.

*** NOTE: If Gitlab complains about pipelines stating that you need to "Validate your Gitlab account" there is no need to worry. You do not have to do this and it is only occurring because OUR pipeline tried to run on your fork. While it's preferred that you do validate your account, you can safely ignore/dismiss this message from Gitlab. ***

Edited by bushtail

Merge request reports