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.

Sign up for GitHub

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

Jump to bottom

Fix faction target logic #473

Merged
merged 1 commit into from
Aug 3, 2022
Merged

Fix faction target logic #473

merged 1 commit into from
Aug 3, 2022

Conversation

dudantas
Copy link
Contributor

@dudantas dudantas commented Aug 1, 2022 •
edited
Loading

The factions were targeting wrong, now everything will work ok
Resolves opentibiabr/otservbr-global#621

@sonarcloud
Copy link

sonarcloud bot commented Aug 1, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@dudantas dudantas mentioned this pull request Aug 1, 2022
Closed
5 tasks
CleberMoreira
CleberMoreira approved these changes Aug 1, 2022
View reviewed changes
Copy link

@CleberMoreira CleberMoreira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and Approved! working as it should, no Lions attackin Players and they attack Usurpers normally.

@dudantas dudantas merged commit 5f70e55 into main Aug 3, 2022
@dudantas dudantas deleted the fix-faction-logic branch August 3, 2022 00:43
luan pushed a commit that referenced this pull request Jul 11, 2023
@dudantas
Fix monsters faction target logic (#473)
8506600 
The factions were targeting wrong, now everything will work ok
Resolves opentibiabr/otservbr-global#621
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guispiller guispiller guispiller approved these changes

@majestyotbr majestyotbr majestyotbr approved these changes

@CleberMoreira CleberMoreira CleberMoreira approved these changes

Assignees
No one assigned
Labels
Area: Source
Projects
Canary Roadmap
Status: PRs Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] monster lion knight, lion warlock
4 participants
@dudantas @guispiller @majestyotbr @CleberMoreira