fix(versioning): use transaction.atomic to prevent corrupt versions being created

[matthewelwell]

Changes

Prevent versions from being persisted to the database if the request fails.

How did you test this code?

Updated an existing test.

[vercel]

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

3 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Ignored (Inspect)			Sep 10, 2024 5:39pm
flagsmith-frontend-preview	⬜️ Ignored (Inspect)			Sep 10, 2024 5:39pm
flagsmith-frontend-staging	⬜️ Ignored (Inspect)			Sep 10, 2024 5:39pm

[github-actions]

Docker builds report

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

[github-actions]

Uffizzi Preview deployment-56110 was deleted.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.16%. Comparing base (1b22cf5) to head (362380c).
Report is 9 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4617   +/-   ##
=======================================
  Coverage   97.16%   97.16%           
=======================================
  Files        1160     1160           
  Lines       40137    40159   +22     
=======================================
+ Hits        39000    39022   +22     
  Misses       1137     1137           

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