Skip to content

Conversation

@Zaimwa9
Copy link
Contributor

@Zaimwa9 Zaimwa9 commented Nov 27, 2025

Thanks for submitting a PR! Please check the boxes below:

  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Contributes to #6130

⚠️ This PR must be merged and deployed right after the deployment of the backend related one that migrates existing feature health provider and updates the choices of Feature Health providers

Changes

  • Updated all occurrences of Sample to Webhook in the code (including payload passed in the create request)
  • Updated documentation

How did you test this code?

@Zaimwa9 Zaimwa9 requested review from a team as code owners November 27, 2025 13:49
@Zaimwa9 Zaimwa9 requested review from kyle-ssg and matthewelwell and removed request for a team November 27, 2025 13:49
@vercel
Copy link

vercel bot commented Nov 27, 2025

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

3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
docs Ignored Ignored Preview Nov 27, 2025 4:35pm
flagsmith-frontend-preview Ignored Ignored Preview Nov 27, 2025 4:35pm
flagsmith-frontend-staging Ignored Ignored Preview Nov 27, 2025 4:35pm

@github-actions github-actions bot added front-end Issue related to the React Front End Dashboard docs Documentation updates labels Nov 27, 2025
@Zaimwa9 Zaimwa9 requested review from talissoncosta and removed request for kyle-ssg November 27, 2025 13:50
@github-actions github-actions bot added feature New feature or request and removed docs Documentation updates labels Nov 27, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 27, 2025

Docker builds report

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

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on December 10

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@Zaimwa9 Zaimwa9 changed the title feat: frontend-rename-feature-health-sample-to-generic ⚠️ feat: frontend-rename-feature-health-sample-to-webhook ⚠️ Nov 27, 2025
@github-actions github-actions bot added docs Documentation updates feature New feature or request and removed feature New feature or request docs Documentation updates labels Nov 27, 2025
talissoncosta
talissoncosta previously approved these changes Nov 27, 2025
Copy link
Contributor

@talissoncosta talissoncosta left a comment

Choose a reason for hiding this comment

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

Looks pretty good! Thanks for handling that!

@github-actions github-actions bot added docs Documentation updates feature New feature or request and removed feature New feature or request docs Documentation updates labels Nov 27, 2025
@Zaimwa9 Zaimwa9 merged commit 6afcf9d into main Dec 2, 2025
40 of 41 checks passed
@Zaimwa9 Zaimwa9 deleted the feat/rename-feature-health-sample-to-webhook branch December 2, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants