Skip to content

fix: revert prevent infinite API loop on invalid environment URLs#7312

Merged
talissoncosta merged 1 commit intomainfrom
revert-7284-fix/environment-ready-checker-infinite-loop
Apr 22, 2026
Merged

fix: revert prevent infinite API loop on invalid environment URLs#7312
talissoncosta merged 1 commit intomainfrom
revert-7284-fix/environment-ready-checker-infinite-loop

Conversation

@talissoncosta
Copy link
Copy Markdown
Contributor

@talissoncosta talissoncosta commented Apr 22, 2026

Reverts #7284

@talissoncosta talissoncosta requested a review from a team as a code owner April 22, 2026 12:29
@talissoncosta talissoncosta requested review from kyle-ssg and removed request for a team April 22, 2026 12:29
claude[bot]
claude Bot reviewed Apr 22, 2026
View reviewed changes
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, reopen this pull request to trigger a review.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 22, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
flagsmith-frontend-preview Ready Ready Preview, Comment Apr 22, 2026 0:30am
flagsmith-frontend-staging Ready Ready Preview, Comment Apr 22, 2026 0:30am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview Apr 22, 2026 0:30am

Request Review

@github-actions github-actions Bot added the front-end Issue related to the React Front End Dashboard label Apr 22, 2026
@talissoncosta talissoncosta requested review from Zaimwa9 and removed request for kyle-ssg April 22, 2026 12:30
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026 •
edited
Loading

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-7312 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-7312 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-api:pr-7312 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith:pr-7312 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-private-cloud:pr-7312 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-frontend:pr-7312 Finished ✅ Results ✅

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026 •
edited
Loading

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  11 passed

Details

stats  11 tests across 8 suites
duration  24.3 seconds
commit  6720461
info  🔄 Run: #16252 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  11 passed

Details

stats  11 tests across 8 suites
duration  52.7 seconds
commit  6720461
info  🔄 Run: #16252 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  17 passed

Details

stats  17 tests across 14 suites
duration  53.1 seconds
commit  6720461
info  🔄 Run: #16252 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  17 passed

Details

stats  17 tests across 14 suites
duration  59.1 seconds
commit  6720461
info  🔄 Run: #16252 (attempt 1)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026

Visual Regression

16 screenshots compared. See report for details.
View full report

@Zaimwa9 Zaimwa9 changed the title Revert "fix: prevent infinite API loop on invalid environment URLs" fix: revert prevent infinite API loop on invalid environment URLs Apr 22, 2026
@talissoncosta talissoncosta merged commit 2eb5f9a into main Apr 22, 2026
29 of 30 checks passed
@talissoncosta talissoncosta deleted the revert-7284-fix/environment-ready-checker-infinite-loop branch April 22, 2026 13:04
@flagsmithdev flagsmithdev mentioned this pull request Apr 22, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@claude claude[bot] claude[bot] left review comments

@Zaimwa9 Zaimwa9 Zaimwa9 approved these changes

Assignees

No one assigned

Labels

front-end Issue related to the React Front End Dashboard

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@talissoncosta @Zaimwa9