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

Move attachments to a new app #1813

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Move attachments to a new app #1813

wants to merge 3 commits into from

Conversation

rafalp
Copy link
Owner

@rafalp rafalp commented Sep 10, 2024 •
edited
Loading

This PR moves attachments to misago.attachments app from misago.threads app.

Fixes #1805 and #1812

TODO

  • Create misago.attachments app
  • Move attachments data from misago.threads
  • Move admin to misago.admin
  • Move attachment views to misago.attachments
  • Add attachment upload view to misago.attachments
  • Use is_deleted flag on attachments for their deletion

@rafalp rafalp added the area: backend This issue involves Python, Django or dependency (eg. database) label Sep 10, 2024
@rafalp rafalp added this to the Next feature release milestone Sep 10, 2024
@rafalp rafalp self-assigned this Sep 10, 2024
@rafalp rafalp added the feature: admin Affects admin control panel label Sep 10, 2024
@rafalp rafalp force-pushed the fix-1805-move-attachemts-to-app branch from 0cc2750 to 96fb0a4 Compare September 15, 2024 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@rafalp rafalp

Labels
area: backend This issue involves Python, Django or dependency (eg. database) feature: admin Affects admin control panel
Projects
None yet
Milestone
Next feature release
Development

Successfully merging this pull request may close these issues.

Move attachments from misago.threads to misago.attachments.
1 participant
@rafalp