Fix TS2300 duplicate import errors in HoverButtons.tsx blocking CI build - #110
Closed
olivhoenen with Copilot wants to merge 2 commits into
Closed
Fix TS2300 duplicate import errors in HoverButtons.tsx blocking CI build#110olivhoenen with Copilot wants to merge 2 commits into
olivhoenen with Copilot wants to merge 2 commits into
Conversation
Documentation build overview
134 files changed ·
|
Copilot
AI
changed the title
[WIP] Fix failing GitHub Actions job build-and-packaging
Fix TS2300 duplicate import errors in HoverButtons.tsx blocking CI build
Jul 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR #101 introduced a merge conflict resolution that left duplicate
import { Configuration, CustomizedGridType, DataGridPlot }statements inHoverButtons.tsx, causing 6TS2300: Duplicate identifierTypeScript errors and failing thebuild-and-packagingCI job.Root cause
When
develop(which had consolidated those three identifiers into a multi-line block also containingPlotType) was merged into the PR branch, both the original single-line import from the PR and the expanded block fromdevelopended up coexisting:Changes
Bug fix
HoverButtons.tsx— Removed the redundant single-line import; retains only the multi-line block (which already covers all needed identifiers includingPlotType). Updates@mantine/hooksimport to adduseElementSizeanduseMergedRef, replaceshoverRef.current.offsetWidthwith reactivecontainerWidthfromuseElementSize.Feature changes from PR #101 (applied cleanly)
CustomizeSmoothing.tsx(new) — UI for Gaussian and Savitzky-Golay filter parameters with apply/restore workflow per selected plotCustomizeUnaryOperations.tsx(new) — UI for composing ordered scalar operations (add/subtract/multiply/etc.) per selected plotutils/fetchData.ts— AddsgetSmoothingMethods,getOperationMethods,buildSmoothingRequest,formatOperations, and smoothing/operations query-string encoding infetchDataPlotutils/plot.ts— AddsreapplyAxisOrderto restore user axis transposition after back-end re-fetches; passessmoothingandoperationsthroughplotNodeUriLoadedtypes/plot.ts— NewSmoothingParamsandUnaryOperationtypes;smoothingandoperationsfields onBaseDataPlotlytypes/data.ts— ExtendedDataManipulationParameterResponsewithfields,group_label, optionalpossible_values;DataManipulationMethodResponsewithadditional_parametersDataplotCustomization.tsx— Registers new Data smoothing and Unary operations accordion panelsCustomizeDownsampling.tsx/CustomizeInterpolation.tsx— CallreapplyAxisOrderafter re-fetch to preserve transpositionMainLayout.tsx— Preservessmoothingandoperationsfields when rebuilding plot objects on config load