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 serialization of components with custom metadata #467

Merged
merged 3 commits into from Nov 16, 2022

Conversation

Scrumplex
Copy link
Member

Prism has introduced a few extensions to the meta component version
format, which were not exported as JSON in the launcher. This caused the
Customize button on the version page to not write these new properties
to the custom component file.

Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
Prism has introduced a few extensions to the meta component version
format, which were not exported as JSON in the launcher. This caused the
`Customize` button on the version page to not write these new properties
to the custom component file.

Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
@Scrumplex Scrumplex added bug Something isn't working simple change labels Nov 16, 2022
@Scrumplex Scrumplex added this to the 6.0 milestone Nov 16, 2022
@Scrumplex Scrumplex changed the title fix: remove unnecessary condition Add serialization of components with custom metadata Nov 16, 2022
@Scrumplex Scrumplex changed the title Add serialization of components with custom metadata Fix serialization of components with custom metadata Nov 16, 2022
launcher/minecraft/MojangVersionFormat.cpp Outdated Show resolved Hide resolved
Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
@Scrumplex Scrumplex merged commit 298871e into PrismLauncher:develop Nov 16, 2022
TheKodeToad added a commit to TheKodeToad/PrismLauncher that referenced this pull request Nov 17, 2022
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working simple change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants