Skip to content

fix(ui): skip API environments calls and SSE stream when user is not authenticated#5670

Merged
kodiakhq[bot] merged 2 commits intov2from
rd/v2/ui-skip-pull-updates-with-no-auth
Apr 5, 2026
Merged

fix(ui): skip API environments calls and SSE stream when user is not authenticated#5670
kodiakhq[bot] merged 2 commits intov2from
rd/v2/ui-skip-pull-updates-with-no-auth

Conversation

@erka
Copy link
Copy Markdown
Contributor

@erka erka commented Apr 4, 2026

Prevents environments query and SSE connection attempts when
authentication is required but user is not logged in.

…authenticated

Prevents environments query and SSE connection attempts when
authentication is required but user is not logged in.

Signed-off-by: Roman Dmytrenko <rdmytrenko@gmail.com>
@erka erka requested a review from a team as a code owner April 4, 2026 20:33
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 4, 2026
@erka erka added automerge Used by Kodiak bot to automerge PRs v2 Flipt v2 labels Apr 4, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.12%. Comparing base (bb25754) to head (5ac3fde).
⚠️ Report is 1 commits behind head on v2.

Additional details and impacted files
@@           Coverage Diff           @@
##               v2    #5670   +/-   ##
=======================================
  Coverage   61.12%   61.12%           
=======================================
  Files         141      141           
  Lines       14170    14170           
=======================================
  Hits         8662     8662           
  Misses       4777     4777           
  Partials      731      731           
Flag Coverage Δ
integrationtests 34.52% <ø> (ø)
unittests 52.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kodiakhq kodiakhq Bot merged commit 1dbb161 into v2 Apr 5, 2026
32 checks passed
@kodiakhq kodiakhq Bot deleted the rd/v2/ui-skip-pull-updates-with-no-auth branch April 5, 2026 14:04
@github-project-automation github-project-automation Bot moved this to Done in Flipt V2 Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Used by Kodiak bot to automerge PRs size:M This PR changes 30-99 lines, ignoring generated files. v2 Flipt v2

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants