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

Improve Photomanagement #244

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Improve Photomanagement #244

wants to merge 10 commits into from

Conversation

ToppAce
Copy link
Contributor

@ToppAce ToppAce commented May 11, 2021

Beschreibung

Mainly closing issue #140

  • All picutures uploaded in the martor texteditor are now saved as Photologue-Photos and thus also deletable by admins.

  • New command to add the host attribute to old imported news posts (addhoststophotos) . This is also necessary to make them deletable by the corresponding host admins.

  • New command (convertmartorphotos) to convert existing photos uploaded by the texteditor to Photologue-Photos to have a consistent data structure (and to make them deletable)

  • Thumbnails are added in the admin list_views of Posts, Projects, Events, Teams, User

Small changes

Typ

  • Bugfix
  • Refactoring
  • Model Changes

Änderungen

Checkliste

  • Dieses PR beinhaltet Model Änderungen (Migrations ausführen)
  • Execute command: manage.py addhoststophotos
  • Execute command: manage.py convertmartorphotos
  • Delete folder of media/images/uploads/ if convertmartorphotos was succesful to make sure that no pictures survive on the server which are not used any more

Gitflow

Assign den PR mindestens 1 Person, gerne mehreren für die Review. Der Request einer Review dient auch zum Informationsaustausch über Änderungen im Code(Migrations, Features etc.). Ein PR muss von mindestens 1 Person reviewed und von dieser in den Master gemerget werden.

@ToppAce ToppAce requested a review from bhemmer May 11, 2021 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant