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

improve: boss cooldown timer #963

Merged
merged 7 commits into from
Mar 31, 2023

Conversation

duuh30
Copy link
Contributor

@duuh30 duuh30 commented Mar 29, 2023

Description

Creating a new function to use in lua to update Boss Cooldown in the Client.

If you use the mechanism created for the Lever() class it takes care of this process automatically.

EXAMPLE

When killing the boss Duke Krule automatically the Cooldown timer will be updated. But if you kill Boss Grand Master Oberon he won't upgrade. This happens because the Oberon Lever update is done manually

NOTE

For manual cases like above example. OBERON it is necessary to use the player:sendBosstiaryCooldownTimer() function after doing setStorageValue in the KEY Timer of the Boss.

Behaviour

When this task is completed, the expected operation is when accessing the boss's room the Cooldown of that boss appears in the client without having to relog.

Actual

It only shows boss cooldowns on login.
If you access the boss X room, it does not update.

Expected

When this task is completed it is expected to update the client's boss cooldown whenever accessing a boss room.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested

To test if the change really works. I used the bosstiary_kill.lua file and added the created function player:sendBoostiaryCooldownTimer() on line 17.

When killing a new boss it is updated in the Bosse Cooldown tab.

Test Configuration:

  • Server Version: 2.6.1
  • Client: 12.91
  • Operating System: Windows

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I checked the PR checks reports
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

@duuh30 duuh30 changed the title Boostiary Cooldown timer | player:sendBoostiaryCooldownTimer() Boostiary Cooldown Timer | player:sendBoostiaryCooldownTimer() Mar 29, 2023
@duuh30 duuh30 changed the title Boostiary Cooldown Timer | player:sendBoostiaryCooldownTimer() Boos Cooldown Timer | player:sendBoostiaryCooldownTimer() Mar 29, 2023
@duuh30 duuh30 changed the title Boos Cooldown Timer | player:sendBoostiaryCooldownTimer() Boos Cooldown Timer | player:sendBossesCooldownTimer() Mar 29, 2023
@duuh30 duuh30 changed the title Boos Cooldown Timer | player:sendBossesCooldownTimer() Boos Cooldown Timer | player:sendBoostiaryCooldownTimer() Mar 29, 2023
@majestyotbr majestyotbr changed the title Boos Cooldown Timer | player:sendBoostiaryCooldownTimer() improve: boss cooldown timer Mar 29, 2023
@duuh30 duuh30 mentioned this pull request Mar 29, 2023
5 tasks
@LeoBorges18 LeoBorges18 mentioned this pull request Mar 30, 2023
1 task
@murilo09 murilo09 marked this pull request as draft March 30, 2023 02:34
@duuh30 duuh30 marked this pull request as ready for review March 30, 2023 03:09
@sonarqubecloud
Copy link

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 merged commit c047a9e into opentibiabr:main Mar 31, 2023
@majestyotbr majestyotbr linked an issue Mar 31, 2023 that may be closed by this pull request
5 tasks
luan pushed a commit that referenced this pull request Jul 11, 2023
Creating a new function to use in Lua to update Boss Cooldown in the Client.

If you use the mechanism created for the Lever() class, it takes care of this process automatically.

Example:
When killing the boss Duke Krule automatically, the Cooldown timer will be updated. But if you kill Boss Grand Master Oberon, he won't upgrade. This happens because the Oberon Lever update is done manually

Note:
For manual cases like above example. Oberon it is necessary to use the player:sendBosstiaryCooldownTimer() function after doing setStorageValue in the key timer of the boss.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update bossTimer
5 participants