Skip to content

Conversation

@parksjr
Copy link
Contributor

@parksjr parksjr commented Dec 17, 2025

Related Issue(s):

  • no-issue

Proposed Changes:

  • Fixes a regression with one of the latest unreleased features for the 7.0 milestone. If the stac server throws an error, the ui would bug out with an infinite loop of retries.

The fix is just adding an error state in redux to prevent the infinite retry. I thought about making this better (loading state, error reset and such) but I'd rather refactor the whole service and manual redux state pattern we have here to use something like RTK query instead. We'll see about that.

  • appends STAC header cookies on get item request

PR Checklist:

  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

@element84-infra
Copy link

element84-infra commented Dec 17, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@parksjr parksjr requested a review from KayakinCoder December 17, 2025 14:33
@parksjr parksjr closed this Dec 17, 2025
@parksjr parksjr deleted the bug/config-breaking-change branch December 17, 2025 19:58
@parksjr parksjr restored the bug/config-breaking-change branch December 17, 2025 21:10
@parksjr parksjr reopened this Dec 17, 2025
@parksjr parksjr force-pushed the bug/config-breaking-change branch from e851ee6 to 498f1a4 Compare December 17, 2025 21:13
@parksjr parksjr changed the title Bug/config breaking change 🐞 stac service regression Dec 18, 2025
@parksjr parksjr added this to the 7.0 milestone Dec 18, 2025
@parksjr parksjr merged commit 4b94e9b into main Jan 6, 2026
3 checks passed
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.

4 participants