Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop using django-multiselect #707

Closed
hmpf opened this issue Dec 6, 2023 · 1 comment · Fixed by #724
Closed

Stop using django-multiselect #707

hmpf opened this issue Dec 6, 2023 · 1 comment · Fixed by #724
Assignees
Labels

Comments

@hmpf
Copy link
Contributor

hmpf commented Dec 6, 2023

It seems to be dead and the forks all seem dead.

We currently use it only for TimeRecurrence.days. We used to use it for NotificationProfile.media.

  1. We'd have to first get rid of the usage
  2. make a migration for that
  3. in a squashing of the migrations of notificationprofile replace all migrations mentioning the library
  4. then in a final migration delete the migration-files importing the field.

Three releases: patch (or minor), patch, minor (or major), the final one drops the dependency.

@johannaengland
Copy link
Contributor

Closed by #711 and #724.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants