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

Upstream Sync for Minecraft 1.13 #6

Merged
merged 23 commits into from
Jun 16, 2024
Merged

Conversation

LostLuma
Copy link
Contributor

@LostLuma LostLuma commented Apr 23, 2024

New features cherry-picked from upstream ModMenu.

Resolves #4.
Resolves #5.

LostLuma and others added 15 commits April 23, 2024 02:45
- Updated Norwegian Bokmål translations
- Updated Chinese translations
- Updated Belarusian translations
- Change Twitter name to X (Twitter)
- Delay loading API implementations until it's first needed in attempt to fix Architectury API's usage (beta)
* Add support for custom update checking logic

* Rename ModrinthUtil to UpdateCheckerUtil

* Make download link required

* Fix search

* Use a separate thread for every custom update checker

---------

Co-authored-by: Max Henkel <max@maxhenkel.de>
- Fixed disable_update_checker config option not working
…r (#704)

* Allow choosing an update channel to receive notifications for

* Pass user preferred update channel to UpdateChecker

* Change method of getting the preferred update channel

* Request preferred version types from Modrinth only

* Switch update channel and quick configure button order

* Rerun update checks when clicking done in the modmenu options screen

- No longer promotes updates to beta or alpha versions by default
* Display credits by role instead of bunching them together

* Add newlines between credit groups, allow translating role names

* Fix minor code style issues

* Improve translation key generation and fallback role names

* Sort contributors within roles

* Fix Fabric authors being credited wrong

- Group mod credits together by role
- Fix Mod Menu build metadata not being properly stripped from the User Agent string generation for privacy
- Add update checkers for Fabric Loader and Quilt Loader
- Add API for mods to provide update checkers for other mods
…her than the button

- Fix update checker not being run when config screen is closed without the Done button
@LostLuma
Copy link
Contributor Author

This should not be merged yet since there is an issue that needs to be fixed upstream (or in Modrinth?) and then added here.

LostLuma and others added 4 commits June 15, 2024 17:17
- Fixed contributors being sorted
- Update Chinese Translation
- Updated Tatar Translation
- Updated Polish Translation
Gazmanovich and others added 3 commits June 15, 2024 17:17
- Updated Belarusian Translation
Fixes #693

- Fixed crash with unknown badges
…(#716)

- Fixed update checker promoting downgrades
@LostLuma
Copy link
Contributor Author

Upstream got some bug fixes merged, meaning the issues with this are now resolved. I will update the other PRs I made for the other branches tomorrow.

@LostLuma
Copy link
Contributor Author

And now that's done!
I've tested the changes in game in the latest patch version of every minor version as well.

Copy link
Member

@SpaceWalkerRS SpaceWalkerRS left a comment

Choose a reason for hiding this comment

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

I didn't look at it too thorougly, if the changes were accepted in upstream that's good enough for me.

@SpaceWalkerRS SpaceWalkerRS merged commit c6195c1 into OrnitheMC:1.13 Jun 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
10 participants