Flagsmith / flagsmith

Open Source Feature Flagging and Remote Config Service. Host on-prem or use our hosted version at https://flagsmith.com/
https://flagsmith.com/
BSD 3-Clause "New" or "Revised" License
4.77k stars 365 forks source link

feat(environment): Add toggle for identity override in local eval #4576

Closed gagantrivedi closed 4 weeks ago

gagantrivedi commented 4 weeks ago

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

Changes

Please describe.

How did you test this code?

Please describe.

vercel[bot] commented 4 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Skipped Deployments | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **docs** | ⬜️ Ignored ([Inspect](https://vercel.com/flagsmith/docs/HjmV6Z6DSUom2tSYMNP7jmN1ut3V)) | [Visit Preview](https://docs-git-feat-identity-override-toggle-flagsmith.vercel.app) | | Sep 4, 2024 10:00am | | **flagsmith-frontend-preview** | ⬜️ Ignored ([Inspect](https://vercel.com/flagsmith/flagsmith-frontend-preview/7fYxNwDt53ja2kUxgWdWpSjjGNJ8)) | [Visit Preview](https://flagsmith-frontend-preview-git-feat-identity-o-b5edd0-flagsmith.vercel.app) | | Sep 4, 2024 10:00am | | **flagsmith-frontend-staging** | ⬜️ Ignored ([Inspect](https://vercel.com/flagsmith/flagsmith-frontend-staging/6HDJhUehftQjTqQix8Bxx7hMoEfW)) | [Visit Preview](https://flagsmith-frontend-staging-git-feat-identity-o-41ae94-flagsmith.vercel.app) | | Sep 4, 2024 10:00am |
sentry-io[bot] commented 4 weeks ago

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: api/util/mappers/engine.py

Function Unhandled Issue
map_environment_to_engine ValidationError: 1 validation error for FeatureStateModel util.mappers.engine in map_featurestate...
Event Count: 1

Did you find this useful? React with a 👍 or 👎

github-actions[bot] commented 4 weeks ago

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-e2e:pr-4576 Finished :white_check_mark: Skipped
ghcr.io/flagsmith/flagsmith-api-test:pr-4576 Finished :white_check_mark: Skipped
ghcr.io/flagsmith/flagsmith-frontend:pr-4576 Finished :white_check_mark: Results :white_check_mark:
ghcr.io/flagsmith/flagsmith-api:pr-4576 Finished :white_check_mark: Results :white_check_mark:
ghcr.io/flagsmith/flagsmith:pr-4576 Finished :white_check_mark: Results :white_check_mark:
ghcr.io/flagsmith/flagsmith-private-cloud:pr-4576 Finished :white_check_mark: Results :white_check_mark:
github-actions[bot] commented 4 weeks ago

Uffizzi Preview deployment-55861 was deleted.

codecov[bot] commented 4 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.14%. Comparing base (8e371a8) to head (bb764e7). Report is 7 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4576 +/- ## ======================================= Coverage 97.14% 97.14% ======================================= Files 1154 1155 +1 Lines 39816 39838 +22 ======================================= + Hits 38679 38701 +22 Misses 1137 1137 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.