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.

Sign up for GitHub

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

Jump to bottom

Deleting media partially fail when already used as post cover #356

Closed
trinity-1686a opened this issue Dec 15, 2018 · 0 comments
Closed

Deleting media partially fail when already used as post cover #356

trinity-1686a opened this issue Dec 15, 2018 · 0 comments
Labels
A: Backend Code running on the server C: Bug Something isn't working

Comments

@trinity-1686a
Copy link
Contributor

Deleting a media that is used as a post cover don't delete the db entry, but delete the file. We should either fail with a message or remove the cover from the post instead.

Plume version (and/or commit): 0.2.0
@trinity-1686a trinity-1686a added C: Bug Something isn't working P: Medium A: Backend Code running on the server labels Dec 15, 2018
trinity-1686a added a commit that referenced this issue Dec 25, 2018
@trinity-1686a
Allow for media cover deletion
6ba3f53 
Fix #356
@trinity-1686a trinity-1686a mentioned this issue Dec 25, 2018
Merged
trinity-1686a added a commit that referenced this issue Jan 5, 2019
@trinity-1686a
Allow for media cover deletion (#387)
aa72334 
* Allow for media cover deletion

Fix #356

* Fix sqlite migrations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A: Backend Code running on the server C: Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@trinity-1686a