Skip to content

Conversation

@modmuss50
Copy link
Member

@modmuss50 modmuss50 commented Mar 25, 2025

  • Get the mod to compile
  • Make sure I didnt break anything when porting
  • Update the docs pages related to the changes
  • appendTooltip is now deprecated, should it be using components?

Please feel free to make PRs into this branch.

@modmuss50 modmuss50 requested review from a team as code owners March 25, 2025 18:59
@netlify
Copy link

netlify bot commented Mar 25, 2025

Deploy Preview for nimble-elf-d9d491 ready!

Name Link
🔨 Latest commit c8d3c45
🔍 Latest deploy log https://app.netlify.com/sites/nimble-elf-d9d491/deploys/67e2fd162c57460008ad9ed2
😎 Deploy Preview https://deploy-preview-316--nimble-elf-d9d491.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@its-miroma its-miroma added todo:verify This needs to be reviewed new-version Related to a newer version of Minecraft priority:low This is nice to have, but it can wait a little priority:medium This should be addressed relatively quickly and removed priority:low This is nice to have, but it can wait a little labels Mar 26, 2025
its-miroma

This comment was marked as resolved.

@its-miroma its-miroma changed the title Port to 1.21.5 Update to 1.21.5 Mar 26, 2025
@its-miroma its-miroma removed the priority:medium This should be addressed relatively quickly label Mar 26, 2025
@IMB11 IMB11 linked an issue Apr 5, 2025 that may be closed by this pull request
@its-miroma its-miroma linked an issue Apr 26, 2025 that may be closed by this pull request
@its-miroma its-miroma added todo:expand This should be expanded priority:high This should be addressed before anything else and removed todo:verify This needs to be reviewed labels May 1, 2025
@its-miroma its-miroma linked an issue Jun 2, 2025 that may be closed by this pull request
Copy link

@TotalSync TotalSync left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only reviewed the gradle updates and anything pertaining to blocks and items. I am not comfortable enough with the rendering to give it the go-ahead. If I get more time, I will circle back and see if I can validate it.

@its-miroma
Copy link
Member

superseded by #347

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-accepted This doesn't look right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants