Add a simple CommaSeparatedValuesField for DB models.

Review Request #13560 — Created Feb. 20, 2024 and submitted

Information

Djblets
release-5.x

Reviewers

We're running into some trouble that the old django-multiselectfield
project is unmaintained and does not support Django 4.x+. There's a
third-party fork which has had some updates, but no releases of that
that we can consume.

django-multiselectfield itself isn't very big, and of what it offers, we
don't actually use very much. This change adds a new field type to
Djblets to do the database serialization/deserialization to join and
split comma-separated values. It doesn't support the validation that
django-multiselectfield does, but we don't actually care about that
functionality.

  • Ran unit tests.
  • Used this in Review Board and tested the functionality of the
    WebHookTarget admin form.
Summary ID
Add a simple CommaSeparatedValuesField for DB models.
We're running into some trouble that the old django-multiselectfield project is unmaintained and does not support Django 4.x+. There's a third-party fork which has had some updates, but no releases of that that we can consume. django-multiselectfield itself isn't very big, and of what it offers, we don't actually use very much. This change adds a new field type to Djblets to do the database serialization/deserialization to join and split comma-separated values. It doesn't support the validation that django-multiselectfield does, but we don't actually care about that functionality. Testing Done: - Ran unit tests. - Used this in Review Board and tested the functionality of the WebHookTarget admin form.
e7c884574cff4811f23eb3f66acb84c4a92858d0
Description From Last Updated

Missing type here and in the docstring.

maubinmaubin

Leftover dev code.

maubinmaubin
david
maubin
  1. 
      
  2. Missing type here and in the docstring.

  3. Leftover dev code.

  4. 
      
david
chipx86
  1. Ship It!
  2. 
      
maubin
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-5.x (4b8e068)
Loading...