Skip to content

Fix Some Sentry Issues - #1026

Merged
Wyvest merged 21 commits into
Polyfrost:v1from
lowercasebtw:v1
Sep 10, 2026
Merged

Fix Some Sentry Issues#1026
Wyvest merged 21 commits into
Polyfrost:v1from
lowercasebtw:v1

Conversation

@lowercasebtw

Copy link
Copy Markdown
Contributor

POLYPLUS3-1XS
POLYPLUS3-29C
POLYPLUS3-29D
POLYPLUS3-29G
POLYPLUS3-1Y2
POLYPLUS3-23T

lowercasebtw and others added 2 commits September 2, 2026 17:26
POLYPLUS3-1XS
POLYPLUS3-29C
POLYPLUS3-29D
POLYPLUS3-29G
POLYPLUS3-1Y2
POLYPLUS3-23T
POLYPLUS3-27M

@lunaynx lunaynx left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Non-blocking, but lf imports 🙏

Comment thread minecraft/src/main/java/org/polyfrost/oneconfig/internal/OneConfigMixinInit.java Outdated
Comment thread minecraft/src/main/java/org/polyfrost/oneconfig/internal/OneConfigMixinInit.java Outdated
Comment thread minecraft/src/main/java/org/polyfrost/oneconfig/internal/OneConfigMixinInit.java Outdated
lowercasebtw and others added 13 commits September 2, 2026 18:09
…compose/SkiaCtx.kt

Co-authored-by: Luna <lunaynx@gmail.com>
…nfigMixinInit.java

Co-authored-by: Luna <lunaynx@gmail.com>
…nfigMixinInit.java

Co-authored-by: Luna <lunaynx@gmail.com>
…nfigMixinInit.java

Co-authored-by: Luna <lunaynx@gmail.com>
…config/v1/ConfigManager.java

Co-authored-by: Luna <lunaynx@gmail.com>
…config/v1/ConfigManager.java

Co-authored-by: Luna <lunaynx@gmail.com>
Resolves the two conflicts introduced by upstream's "derive settings panel
state from selection" refactor and the PolyComposeClock split:

- HudDesignStudio: panelHud is now derived from selectedHuds/panelOpen, so
  the stale-HUD delete path no longer clears it by hand and OpenSettings
  sets panelOpen instead.
- PolyComposeHost: the re-entrancy guard and Snapshot.global wrapper move
  onto PolyComposeClock.frame so both the hud and preview clocks get them.
apiCheck fails on upstream/v1 as-is: d5e4a50 added OneConfigHudWrapper
.ownsPlacement and bc83fc3 added HudEditorToggleEvent.SCREEN_REMOVED and
.screenAlreadyGone, but neither re-ran apiDump. Regenerated; the four
entries below are the only change.
lowercasebtw added a commit to lowercasebtw/OneConfig that referenced this pull request Sep 8, 2026
Resolves the two conflicts introduced by upstream's "derive settings panel
state from selection" refactor and the PolyComposeClock split:

- HudDesignStudio: panelHud is now derived from selectedHuds/panelOpen, so
  the stale-HUD delete path no longer clears it by hand and OpenSettings
  sets panelOpen instead.
- PolyComposeHost: the re-entrancy guard and Snapshot.global wrapper move
  onto PolyComposeClock.frame so both the hud and preview clocks get them.
@lowercasebtw
lowercasebtw force-pushed the v1 branch 2 times, most recently from f52a3cd to 241332e Compare September 8, 2026 02:41
lowercasebtw and others added 6 commits September 10, 2026 01:41
72, 71, 7C, 7A, 77, 7D, DS, T0, 9E, 1XR, 1B1, 29G, 29D, 29C, 2AR, 10B, 17R, 27M, BF, KC, 9, 10
# Conflicts:
#	minecraft/src/main/kotlin/org/polyfrost/oneconfig/internal/compat/SkyblockerCompat.kt
#	minecraft/src/main/kotlin/org/polyfrost/oneconfig/internal/ui/compose/SkiaCtx.kt
@Wyvest
Wyvest merged commit 646c512 into Polyfrost:v1 Sep 10, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants