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

Backend: graph editor fix, title logic change #3198

Merged
merged 2 commits into from
Jan 10, 2025
Merged

Conversation

hannibal002
Copy link
Owner

@hannibal002 hannibal002 commented Jan 10, 2025

What

fixed error and deprecated old function

Stack Trace
SkyHanni 1.3.0: Caught a NoSuchElementException in GraphEditor at LorenzTickEvent: Collection contains no element matching the predicate.
 
Caused by java.util.NoSuchElementException: Collection contains no element matching the predicate.
at SH.test.graph.GraphEditor.calculateNewAllNodeFind(GraphEditor.kt:920)
at SH.test.graph.GraphEditor.save(GraphEditor.kt:566)
at SH.test.graph.GraphEditor.input(GraphEditor.kt:418)
at SH.test.graph.GraphEditor.onTick(GraphEditor.kt:206)
at SH.data.MinecraftData.onTick(MinecraftData.kt:84)
at FML.common.eventhandler.EventBus.post(EventBus.java:140)

Changelog Technical Details

  • Fixed bug in /shgraphfindall. - hannibal2

@hannibal002 hannibal002 added Soon This Pull Request will be merged within the next couple of betas Backend A backend pull request that will be merged soon labels Jan 10, 2025
@hannibal002 hannibal002 added this to the Version 2.0.0 milestone Jan 10, 2025
@hannibal002 hannibal002 merged commit bab4144 into beta Jan 10, 2025
11 checks passed
@github-actions github-actions bot removed Soon This Pull Request will be merged within the next couple of betas Backend A backend pull request that will be merged soon labels Jan 10, 2025
@CalMWolfs CalMWolfs deleted the title-and-graph-editor branch January 16, 2025 23:13
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