Skip to content

Conversation

@Kenpachi2k13
Copy link
Member

Pull Request Prelude

Changes Proposed

Guild skills will be ignored in skill_cooldown_save() if guild_skill_relog_delay is set to 1.
If guild skills are not ignored when guild_skill_relog_delay is set to 1, the map server will throw error messages (no such timer) if the guild master relogs while cooldown.
Also applied Hercules coding style to skill_cooldown_save().

Issues addressed: #2591

…relog_delay is set to 1.

If guild skills are not ignored when guild_skill_relog_delay is set to 1, the map server will throw error messages if the guild master relogs while cooldown.
Also applied Hercules coding style to skill_cooldown_save().
See HerculesWS#2591
@Kenpachi2k13 Kenpachi2k13 added type:bug Issue is a bug or describes an incorrect behavior that should be fixed component:core Affecting the Hercules core (i.e. not the game mechanics directly) status:code-review Awaiting code review labels Dec 1, 2019
@Kenpachi2k13 Kenpachi2k13 requested review from 4144 and Asheraf December 1, 2019 05:54
@Kenpachi2k13 Kenpachi2k13 removed the request for review from Asheraf December 14, 2019 01:30
@Kenpachi2k13 Kenpachi2k13 added codereview:accepted Code review was positive and the pull request can be accepted as is and removed status:code-review Awaiting code review labels Dec 14, 2019
@Kenpachi2k13 Kenpachi2k13 added this to the Release v2019.12.15 milestone Dec 14, 2019
@MishimaHaruna MishimaHaruna merged commit 22227b9 into HerculesWS:master Dec 15, 2019
@Kenpachi2k13 Kenpachi2k13 deleted the issue#2591 branch December 31, 2019 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codereview:accepted Code review was positive and the pull request can be accepted as is component:core Affecting the Hercules core (i.e. not the game mechanics directly) type:bug Issue is a bug or describes an incorrect behavior that should be fixed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants