DeluxeMenus only supports Minecraft 1.13.2 now, which means data can no longer be used as data but just as damage.
Instead of continuing to use the Deprecated API feature to set the damage, we can use the new API (Damageable)
I recommend the following steps:
Add a new config option called damage while keeping the old data
If data is present in the config, send a warning in the console letting people know it is being renamed to damage
If both options are present in the config, ignore data
Use the Damageable API to set the damage now
No longer use the deprecated way of setting data
After these changes are completed and merged, update the wiki to reflect them.
At a later stage, completely remove the data option.
Remember to open a new issue regarding the complete removal of the data option after this issue is closed!
DeluxeMenus only supports Minecraft 1.13.2 now, which means
datacan no longer be used as data but just as damage. Instead of continuing to use the Deprecated API feature to set the damage, we can use the new API (Damageable)I recommend the following steps:
damagewhile keeping the olddatadatais present in the config, send a warning in the console letting people know it is being renamed todamagedataAfter these changes are completed and merged, update the wiki to reflect them. At a later stage, completely remove the
dataoption.Remember to open a new issue regarding the complete removal of the
dataoption after this issue is closed!