fix: return-empty-trait-list-if-persistence-not-allowed-from-sdk

[Zaimwa9]

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

Changes

• Validate returns empty traits list

How did you test this code?

• Added tests

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 29, 2025 11:43am
flagsmith-frontend-preview	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 29, 2025 11:43am
flagsmith-frontend-staging	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 29, 2025 11:43am

[sentry]

🔍 Existing Issues For Review

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

📄 File: api/environments/sdk/serializers.py

Function	Unhandled Issue
save	ValueError: Cannot compare feature states as they are missing environment_feature_version. ... Event Count: 4

---

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

[github-actions]

Docker builds report

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

[github-actions]

Uffizzi Ephemeral Environment deployment-63255

☁️ https://app.uffizzi.com/github.com/Flagsmith/flagsmith/pull/5399

📄 View Application Logs etc.

What is Uffizzi? Learn more!

[github-actions]

Uffizzi Preview deployment-63255 was deleted.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.62%. Comparing base (0cfe793) to head (0bbb04c).
Report is 8 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5399   +/-   ##
=======================================
  Coverage   97.62%   97.62%           
=======================================
  Files        1238     1238           
  Lines       43000    43031   +31     
=======================================
+ Hits        41977    42008   +31     
  Misses       1023     1023           

☔ 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.