-
Notifications
You must be signed in to change notification settings - Fork 421
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
[Bots] Bot Overhaul #4580
base: master
Are you sure you want to change the base?
[Bots] Bot Overhaul #4580
Conversation
Add ^setassistee command to set who your bots will assist. Bots will always assist you first before anyone else. If the rule Bots, AllowCrossGroupRaidAssist is enabled bots will assist the group or raid main assists. Rewrites logic in handling of pull and returning to ensure bots make it back to their location.
Previously level requirement was only being checked on the initial rank of an AA. If passed, bots would gain all ranks for that AA regardless of level, this will now check for the level requirement for each rank before granting the AA
const uint32 SPELL_TYPE_MIN = (SpellType_Nuke << 1) - 1; | ||
const uint32 SPELL_TYPE_MAX = (SpellType_PreCombatBuffSong << 1) - 1; | ||
const uint32 SPELL_TYPE_ANY = 0xFFFFFFFF; | ||
namespace BotSpellTypes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alignment issues in this file
@@ -93,6 +93,28 @@ struct AppearanceStruct { | |||
uint8 texture = UINT8_MAX; | |||
}; | |||
|
|||
struct BotSpellSettings_Struct |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alignment issues
std::vector<std::string> description = | ||
{ | ||
"Threshold of your own health when bots will start casting the chosen spell type" | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this command that appears to be bot specific not in the bot gm command section ?
zone/gm_commands/spell_holds.cpp
Outdated
{ | ||
"Toggles whether or not bots can cast certain spell types on you" | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few commands that appear to be bot specific that are not in the bot section
Have a handful of things to round out before I get back to this giant beast. Ideally we can get this wrapped up this week |
Description
Spells:
Spell lists:
Commands:
Movement:
Combat:
General:
Misc:
Probably missed quite a few changes and there is a LOT more detail that has been done here but if bots were previously version 1.0, consider these version 10.0
Type of change
Please delete options that are not relevant.
Testing
Thorough testing done during rewrite. Inevitable with something this big that SOMETHING will pop up and will be addressed as discovered.
Clients tested:
RoF2
Checklist