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

Waypoints API & Custom Dungeon Secrets #390

Merged
merged 10 commits into from Oct 30, 2023

Conversation

kevinthegreat1
Copy link
Collaborator

Custom dungeon secret waypoints!

Use with /skyblocker dungeons secrets addWaypoint|addWaypointRelatively [pos] [secretIndex] [category] [name] and /skyblocker dungeons secrets removeWaypoint|removeWaypointRelatively [pos]


Adds Waypoint abstraction, currently used by Dungeon Secrets and Mythological Ritual.
Fairy Souls, Effigy Waypoints, Enigma Souls, and Relics need to be refactored to use this abstraction.

This provides the groundwork for a more general Waypoints system!

@kevinthegreat1 kevinthegreat1 added the reviews needed This PR needs reviews label Oct 28, 2023
@AzureAaron AzureAaron added the new feature This issue or PR is a new feature label Oct 28, 2023
Copy link
Collaborator

@AzureAaron AzureAaron left a comment

Choose a reason for hiding this comment

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

works great!

@AzureAaron AzureAaron added merge me please Pull requests that are ready to merge and removed reviews needed This PR needs reviews labels Oct 28, 2023
@kevinthegreat1 kevinthegreat1 merged commit ed5957c into SkyblockerMod:master Oct 30, 2023
1 check passed
@kevinthegreat1 kevinthegreat1 deleted the waypoint branch October 30, 2023 04:20
@AzureAaron AzureAaron removed the merge me please Pull requests that are ready to merge label Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature This issue or PR is a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants