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

Prevent repeating kill trips when slayer task is finished #6213

Merged
merged 5 commits into from
Dec 27, 2024

Conversation

DayV-git
Copy link
Contributor

Description:

Add underlying onTask boolean to check if repeated monster killing trip is off task, but original trip was on task. If so, don't do the trip. Can then either get new task, or just use /k again to kill the same monster if you really want to. There were other options, like a confirmation message, but probably unnecessary.
#6185

Closes #5972.

Changes:

  • Add onTask boolean to MonsterActivityTaskOptions and minionKCommand, the latter only accessed via repeat trip.
  • Return fail message when current slayer task has finished but trying to repeat kill the same monster.

Other checks:

  • I have tested all my changes thoroughly.

@gc
Copy link
Collaborator

gc commented Nov 25, 2024

Did people agree/like this idea?

@DayV-git
Copy link
Contributor Author

DayV-git commented Nov 25, 2024

Did people agree/like this idea?

Poll is pretty clear. The bot used to have this functionality but it was removed (unintentionally) a while ago, so to me it was a no-brainer.

image

@gc gc merged commit 1cc856e into oldschoolgg:master Dec 27, 2024
4 checks passed
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.

Prevent repeat trip button working when slayer task is finished
2 participants