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

BSO - added range cape check to reduce arrow & dart cost #3843

Open
wants to merge 3 commits into
base: bso
Choose a base branch
from

Conversation

lordzuku
Copy link

Description:

added master range cape effect to work on zuk

Changes:

passed a boolean for if a master range cape is worn to the cost object, checked if true & then modified the cost per hour for darts & arrows

Other checks:

@6enny6
Copy link
Contributor

6enny6 commented Apr 29, 2022

should this be added to src/lib/data/similaritems.ts ?

make ranger master act as an assembler

@lordzuku
Copy link
Author

lordzuku commented Apr 29, 2022

should this be added to src/lib/data/similaritems.ts ?

make ranger master act as an assembler

iirc the range max cape shouldnt act as an assembler, otherwise the ammo saving bonuses might stack, would probably need it coded into future commands as a separate ammo saving systems
a solution could be to create a user.ammoSaving object or something, & have the object return the highest ammo saving equipped

@anthonyburrow
Copy link

would probably need it coded into future commands as a separate ammo saving systems

I believe this already exists in the "specialRemoveItems" function which is called in inferno.ts, however it only accounts for the Assembler:

public async specialRemoveItems(this: User, _bank: Bank) {

I would probably recommend making this function more generalized for BSO additions by adding the Ranged Master Cape percentage in this cost function instead of making a new condition for just the inferno.

@lordzuku
Copy link
Author

lordzuku commented Apr 29, 2022

would probably need it coded into future commands as a separate ammo saving systems

I believe this already exists in the "specialRemoveItems" function which is called in inferno.ts, however it only accounts for the Assembler:

public async specialRemoveItems(this: User, _bank: Bank) {

I would probably recommend making this function more generalized for BSO additions by adding the Ranged Master Cape percentage in this cost function instead of making a new condition for just the inferno.

unless im mistaken (which i might be, just recently started looking over the project)
that would only apply for regular m k trips & would still need a manual patch for fixing the inferno due to the check that the ammo must be equipped, i assume that check is to fix some kind of issues

currently there is an already built system for m k trips for range max cape saving

prettier/checkstyle formatting
@lordzuku lordzuku changed the title added range cape check to reduce arrow & dart cost BSO - added range cape check to reduce arrow & dart cost May 1, 2022
@osrsjobbo osrsjobbo added the BSO Bot School Old label Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BSO Bot School Old
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants