Added Island Level Configuration #818
Conversation
sh0inx
commented
Mar 1, 2024
sh0inx
commented
- the other side of IridiumTeams #49
- the other side of [IridiumTeams #49](Iridium-Development/IridiumTeams#49)
- added ``minLevel`` to ``BiomeItem`` and ``SchematicConfig`` classes, and functionality to use them - added ``islandCreationCost`` in config to check if we want islands to cost something on creation or just regen - updated configs to reflect changes in teams (also updated default regenCost to be 0 because... y'know) - removed formulas from ``getLevel()`` because we pull from islandManager instead - added ``island_experienceToLevelUp`` and ``island_experienceForNextLevel`` placeholders, which return the amount of experience left needed to level up and the experience needed for the next level respectively
| @@ -56,6 +56,7 @@ public Configuration() { | |||
| public boolean clearEnderChestOnRegen = false; | |||
| public boolean allowPvPOnIslands = false; | |||
| public boolean islandCreateOnJoin = false; | |||
| public boolean islandCreationCost = false; | |||
There was a problem hiding this comment.
I feel like this is going to confuse things even more. I understand there are alot of people asking this in the Discord, but if we want to remove the creation cost, I feel setting the default cost to 0 for everything is a better aproach
There was a problem hiding this comment.
this was more for the fact that we could allow pricing for creation and regen to be different (as in, when you create the island, it doesn't cost anything, but when you regen, it does) - we could change it to be free but honestly i dont care about those people lol
| @@ -61,7 +62,7 @@ public void addContent(Inventory inventory) { | |||
|
|
|||
| private List<Placeholder> getBiomeLorePlaceholders(Biomes.BiomeItem item) { | |||
| List<Placeholder> placeholders = new ArrayList<>(Arrays.asList( | |||
| new Placeholder("amount", IridiumSkyblock.getInstance().getBiomeManager().formatPrice(item.defaultAmount)), | |||
| new Placeholder("minLevel", String.valueOf(item.minLevel)), | |||
There was a problem hiding this comment.
Do we not need the Amount placeholder anymore?
There was a problem hiding this comment.
yes, this allows us to fetch the price of the schematic from the config directly without having to set the lore manually every time
src/main/java/com/iridium/iridiumskyblock/managers/BiomeManager.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Peaches_MLG <[email protected]>
