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

[Pull Request] Small addon to the gringott's config #168

Merged
merged 4 commits into from
Jan 7, 2024

Conversation

iomatix
Copy link

@iomatix iomatix commented Dec 25, 2023

In the configuration files, a new option called custommodeldata-only has been added. By default, it is set to false.
This option determines the type of currency checking used in the system.

There are two types of currency checking available:

Traditional Method:

If custommodeldata-only is set to false (the default), the system will use the traditional method for currency checking.

Alternative Method using Material-CustomModelData Key-Value Pairs:

If custommodeldata-only is set to true, the system will use an alternative method for currency checking.
In this approach, the system looks at the key-value pairs of material-custommodeldata to determine currency.

This update provides flexibility in choosing the currency checking method based on your preferences or specific needs.

…onfiguration files, describing the type of currency checking—either the traditional method or the alternative one using key-value of material-custommodeldata.
…onfiguration files, describing the type of currency checking—either the traditional method or the alternative one using key-value of material-custommodeldata.
…onfiguration files, describing the type of currency checking—either the traditional method or the alternative one using key-value of material-custommodeldata.
…onfiguration files, describing the type of currency checking—either the traditional method or the alternative one using key-value of material-custommodeldata.
Copy link
Author

@iomatix iomatix left a comment

Choose a reason for hiding this comment

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

Works okay at build.

@nikosgram
Copy link
Owner

I think the CI system of Gringotts might be broken or something. I will merge and test it locally :)

@nikosgram nikosgram merged commit 9ace3f6 into nikosgram:master Jan 7, 2024
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.

2 participants