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

Remove Song.restricted field #729

Closed
5 tasks done
BeritJanssen opened this issue Jan 23, 2024 · 0 comments · Fixed by #738
Closed
5 tasks done

Remove Song.restricted field #729

BeritJanssen opened this issue Jan 23, 2024 · 0 comments · Fixed by #738
Assignees

Comments

@BeritJanssen
Copy link
Collaborator

BeritJanssen commented Jan 23, 2024

Right now, we have an option to restrict playback of a section, defined on its associated song, to specific countries. We can actually remove this field altogether, as it's usually allowed to play copy-righted songs for academic purposes. This would mean the following:

  • remove the field restricted from Song model and add a migration
  • update the playlist.update_sections function and its test to not expect that csv field anymore
  • update the hint under the Playlist csv TextField in the admin interface
  • update the csv upload / download functionality accordingly
  • remove all utility functions which make use of the restricted notion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants