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

[Bots] Place BOT_COMMAND_CHAR inside messages #3027

Merged
merged 3 commits into from Mar 5, 2023

Conversation

trentdm
Copy link
Contributor

@trentdm trentdm commented Mar 5, 2023

Fixing an issue with bot portal/circle lists not sending the correct command when the player interacts. The bot command char, ^, was not being properly formatted to a string.

image

@Kinglykrab Kinglykrab marked this pull request as ready for review March 5, 2023 00:42
@Kinglykrab
Copy link
Contributor

Fixed this by just putting the ^ character in the string itself, no reason to build it on to it when we don't do this in most other places.

@Kinglykrab Kinglykrab changed the title Fix bot depart list [Fix] Place BOT_COMMAND_CHAR inside messages Mar 5, 2023
@Akkadius Akkadius changed the title [Fix] Place BOT_COMMAND_CHAR inside messages [Bots] Place BOT_COMMAND_CHAR inside messages Mar 5, 2023
@Kinglykrab Kinglykrab merged commit 034feb4 into EQEmu:master Mar 5, 2023
@Aeadoin Aeadoin mentioned this pull request Mar 24, 2023
nytmyr pushed a commit to nytmyr/Server that referenced this pull request Dec 1, 2023
* Fix bot depart list

* Update bot_command.cpp

* Update bot_command.cpp

---------

Co-authored-by: Alex King <89047260+Kinglykrab@users.noreply.github.com>
Co-authored-by: Kinglykrab <kinglykrab@gmail.com>
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.

None yet

3 participants