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.

Sign up for GitHub

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

Jump to bottom

Create dynamically coded solution for upgrade sound and message for all Upgrade objects. #528

Closed
wants to merge 17 commits into from
Closed

Create dynamically coded solution for upgrade sound and message for all Upgrade objects. #528

wants to merge 17 commits into from

Conversation

oschwa
Copy link

@oschwa oschwa commented Sep 6, 2024

Uses previously mentioned static fields for PlaySoundAction and ChatMessageAction in UpgradeManager, these are generic upgrade output to be used across Upgrade objects.

Other changes to existing code:

  • LevelData uses an additional constructor for sounds and the chat message specifically.
  • AbstractCondition uses a lombok @Getter annotation to add an action to the actions list.
  • LandLevelledUpgrade uses boolean parameters for LevelData's new constructor, removing the need for repeated code.

OwnerAli and others added 17 commits August 29, 2024 14:54
@OwnerAli
add bank and resource system
d50b9ad
@oschwa
Create scoreboard for player.
758d409
@OwnerAli
add scoreboard for resources
d3eae0a
@oschwa
Add listener for player getting resources.
86848a2
@OwnerAli
- add scoreboard updates when a user acquires resources
a796462 
- add packet listeners to remove scoreboard score numbers
- add increment amount for resources (in case we want to have multipliers or something)
@oschwa
Delete src/main/resources/blueprints/island.blu
5d5f21d 
File is a corrupted .blu file.
@oschwa
Add new island.blu file.
f79c456 
Uncorrupted raw file from BSkyBlock.
@OwnerAli
- add 45x45 plot blueprint for starter plot
174eef6
@OwnerAli
[WIP] Add new system to calculate accessible land for player
aa45f0f
@OwnerAli
block players from moving/placing/breaking outside their accessible land
bb45586
@OwnerAli
- add basic upgrade system with levelling type upgrades
8839f26 
- add conditions system with resource condition implementation to decide if a player can unlock upgrade or not
- add action system to execute actions to the factory player with remove resource implementation to remove resources from FactoryPlayer
@OwnerAli
- add description to upgrades to display on gui items
d084c8f
@OwnerAli
- recode entire levelled upgrade system
a86ea03 
- add sound & chat actions
- update gui item for upgrades
@OwnerAli
- update method and class names for PlayerMoveListener.java
382d587
@oschwa
Create level 3 upgrade for player land space.
f2be66f
@OwnerAli
- begin implementing generators
d94469a
@oschwa
Create dynamically-coded solution for upgrade sound and chat message …
bcfb4e0 
…for all Upgrade implementations.
@oschwa oschwa closed this Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oschwa @OwnerAli