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

Added config options for Team Leveling #49

Merged
merged 18 commits into from
Jun 10, 2024
Merged

Added config options for Team Leveling #49

merged 18 commits into from
Jun 10, 2024

Conversation

sh0inx
Copy link
Contributor

@sh0inx sh0inx commented Mar 1, 2024

  • added giveLevelRewards to determine if level rewards are given out for leveling up.
  • added isLevelExponential to determine whether the plugin uses an exponential function when determining the level of the team.
  • added flatExpRequirement to change how much experience is required per level.
  • added curvedExpModifier to determine how much experience is required per level using the exponential function.
  • made getLevel() abstract so that the plugin can use the configuration to determine the level.

- added ``giveLevelRewards`` to determine if level rewards are given out for leveling up.
- added ``isLevelExponential`` to determine whether the plugin uses an exponential function when determining the level of the team.
- added ``flatExpRequirement`` to change how much experience is required per level.
- added ``curvedExpModifier`` to determine how much experience is required per level using the exponential function.
- made ``getLeve()`` abstract so that the plugin can use the configuration to determine the level.
sh0inx added a commit to Iridium-Development/IridiumSkyblock that referenced this pull request Mar 1, 2024
- added ``level`` command, to allow players to check their current team level and to see how much experience they need to level up
- removed ``level`` as an alias from ``info`` command
- removed ``isLevelExponential``, as setting ``curvedModifier`` to 0 will turn it into a linear function.
- added ``levelInfo`` for ``level`` command
- added ``minLevel`` to ``MissionData`` and ``ShopItem`` classes, and functionality to check if the team interacting with these features is the right level to use them.
- replaced constants with variables from the config to allow customization of the level formula (and engineered a formula to get the experience for a level, you're welcome nerds)
- updated tests
@sh0inx sh0inx marked this pull request as ready for review March 16, 2024 14:15
@sh0inx sh0inx mentioned this pull request Apr 30, 2024
sh0inx added a commit to Iridium-Development/IridiumSkyblock that referenced this pull request May 10, 2024
commit ec3d514
Merge: 28800ec 898bc8a
Author: Shyanne <randomgirlisweird@gmail.com>
Date:   Sat Mar 16 10:17:24 2024 -0400

    Merge branch 'master' into level-configuration

commit 28800ec
Author: Shyanne <randomgirlisweird@gmail.com>
Date:   Sun Mar 3 13:26:48 2024 -0500

    Fixed bank placeholders and GUIs

commit c6348a5
Author: Shyanne <randomgirlisweird@gmail.com>
Date:   Sun Mar 3 01:46:14 2024 -0500

    small fixes

commit faaf0c9
Author: Shyanne <randomgirlisweird@gmail.com>
Date:   Sun Mar 3 00:55:32 2024 -0500

    Added More Level Features

    - 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

commit 1c3f686
Author: Shyanne <randomgirlisweird@gmail.com>
Date:   Fri Mar 1 13:07:12 2024 -0500

    added leveling toggle

commit ade8705
Author: Shyanne <randomgirlisweird@gmail.com>
Date:   Fri Mar 1 13:02:00 2024 -0500

    no negative values here

commit c3cc083
Author: Shyanne <randomgirlisweird@gmail.com>
Date:   Fri Mar 1 12:56:12 2024 -0500

    Added Island Level Configuration

    - the other side of [IridiumTeams #49](Iridium-Development/IridiumTeams#49)
Copy link
Member

@PeachesMLG PeachesMLG left a comment

Choose a reason for hiding this comment

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

Mostly look good, just some small things

@PeachesMLG PeachesMLG merged commit a5bba26 into master Jun 10, 2024
2 checks passed
@PeachesMLG PeachesMLG deleted the level-update branch June 10, 2024 16:03
PeachesMLG added a commit to Iridium-Development/IridiumSkyblock that referenced this pull request Jun 10, 2024
* Added Island Level Configuration

- the other side of [IridiumTeams #49](Iridium-Development/IridiumTeams#49)

* no negative values here

* added leveling toggle

* Added More Level Features

- 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

* small fixes

* Fixed bank placeholders and GUIs

* switch to > instead of != for level min

Co-authored-by: Peaches_MLG <PeachesMLG100@gmail.com>

* Changes

---------

Co-authored-by: Peaches_MLG <PeachesMLG100@gmail.com>
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