-
Notifications
You must be signed in to change notification settings - Fork 6
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
apps/activities: prepare removal of activities app #5374
base: main
Are you sure you want to change the base?
Conversation
- migration to delete models - delete all files which can already be safely removed partly fixes #3600
### Removed | ||
|
||
- **Breaking Change** prepare the removal of the activities (face2face) app and delete the | ||
associated models and views |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this breaking if there is no activities happened/happening? And reminds me that I don't have an account on mB prod.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use the app but as we are open source someone else could ? (not that it is very likely but I thought it would still be nice to mention it as a breaking change)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to remove the app from the INSTALLED_APPS
too.
And do we include it in the coming release?
We can include it now I'd say and once it was deployed to prod we can remove it from the |
partly fixes #3600
Tasks