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

feat: random monsters spawn #1802

Merged
merged 8 commits into from
Nov 13, 2023
Merged

Conversation

Schiffers
Copy link
Contributor

By using this commit you can choose if you want a "dinamic" respawn
since it check the monsters at the same spawn
RME imagem ...

teste random spawn

by using the commit
image

the only thing you need care, if is a boss at the same spawn of others monsters, it can duplicate the boss...

just add in config.lua

randomMonsterSpawn = true

plus:

if you do not use it, by default randomMonsterSpawn is false

@dudantas dudantas changed the title Spawn diferenciado feat: random monsters spawn Nov 11, 2023
Copy link

sonarcloud bot commented Nov 13, 2023

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

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

@dudantas dudantas merged commit ca1d489 into opentibiabr:main Nov 13, 2023
25 checks passed
elsongabriel pushed a commit to elsongabriel/canary that referenced this pull request Nov 13, 2023
By using this commit you can choose if you want a "dinamic" respawn since it checks the monsters at the same spawn.
dudantas added a commit that referenced this pull request Nov 14, 2023
dudantas added a commit that referenced this pull request Nov 15, 2023
Reverts #1802

Reason: the change caused bugs in the spawn system.
luan added a commit that referenced this pull request Dec 9, 2023
Rework of #1802 to fix issues with boss spawns and adding a "weight" to
enable users to balance things.

Credits to @Schiffers for the original PR and idea.
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.

2 participants