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

Fix monster loot items stages display on bestiary #24

Merged
merged 7 commits into from
Nov 2, 2021

Conversation

ArturKnopik
Copy link
Contributor

https://tibia.fandom.com/wiki/Cyclopedia#Bestiary
Image show how its work, before this fix all loot is added without regard to the monster discovery level
Missing: very rare loot not being discovered when droped by monster, instead i move the thresholds for uncommon, semi rare, very rare will be fully discovered on complete bestiary
bestiary

@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

No Coverage information No Coverage information
No Duplication information No Duplication information

@majestyotbr majestyotbr changed the title Bestiary monster loot Fix monster loot items stages display on bestiary Aug 31, 2021
@Costallat Costallat force-pushed the master branch 4 times, most recently from 70cfa99 to ccd27ed Compare September 24, 2021 18:26
@Costallat Costallat force-pushed the master branch 20 times, most recently from 29e30e0 to 55e6789 Compare October 5, 2021 17:14
@Costallat Costallat force-pushed the master branch 8 times, most recently from 88b2e44 to 4ab42fc Compare October 6, 2021 20:09
@Bipbop69 Bipbop69 mentioned this pull request Oct 9, 2021
8 tasks
@Methemia Methemia mentioned this pull request Oct 15, 2021
4 tasks
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 2, 2021

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 3 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

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.

4 participants