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 crash on version 1.20 #77

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jbazant
Copy link

@jbazant jbazant commented Nov 6, 2023

Fixes #75.

Proposed Changes

  • fixed access to unitialized BIOMES item
  • Small refactoring of relevant code

@AmonBune AmonBune mentioned this pull request Jan 25, 2024
@RedstoneFuture
Copy link

RedstoneFuture commented Feb 25, 2024

The PR works well on my server. Thanks!

Tested with PaperMC 1.20.4 (build 409) and Dynmap v. 3.7-beta-4-935

grafik

@Pezmc
Copy link

Pezmc commented Aug 22, 2024

For those less technically minded, I've built a release jar which includes this change (thanks go to @jbazant) and some version upgrades, and can be downloaded at https://github.com/Pezmc/dynmap-structures/releases/tag/2.6.2

@UplandJacob
Copy link

For those less technically minded, I've built a release jar which includes this change (thanks go to @jbazant) and some version upgrades, and can be downloaded at https://github.com/Pezmc/dynmap-structures/releases/tag/2.6.2

Amazing! Still works on 1.21.1.

@NODeeJay
Copy link

Yep, but sadly no on 1.21.4 anymore.

1 similar comment
@NODeeJay
Copy link

Yep, but sadly no on 1.21.4 anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not loading in 1.20
5 participants