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

[8306] delete more apps and drop a4 tables #2690

Merged
merged 4 commits into from
Oct 22, 2024
Merged

[8306] delete more apps and drop a4 tables #2690

merged 4 commits into from
Oct 22, 2024

Conversation

goapunk
Copy link
Contributor

@goapunk goapunk commented Sep 12, 2024 •
edited
Loading

  • delete all apps which had migrations deleting their models in the previous PR
  • manual migration to drop all a4 tables
  • prepare removal of organisation app
  • reset migrations for home app as they depend on other migrations (see commit on how to fix migration state manually)

This PR needs to be merged after #2689 was deployed!

This was referenced Sep 12, 2024
Merged
Merged
@goapunk goapunk changed the title [8306] delete more apps and drop a4 tables WIP: [8306] delete more apps and drop a4 tables Sep 16, 2024
Base automatically changed from jd-2024-09-delete-opin-new to main October 21, 2024 10:18
@goapunk goapunk changed the title WIP: [8306] delete more apps and drop a4 tables [8306] delete more apps and drop a4 tables Oct 21, 2024
@goapunk
delete apps
b025621
@goapunk
delete a4 tables, organisation models and reset home migrations.
45eecc6 
run 'python manage.py migrate --fake home zero' and 'python manage.py migrate --fake-initial' to fix the migration state
@goapunk
remove more unused stuff
12c0976
@goapunk goapunk force-pushed the jd-2024-09-delete-opin-apps-part2 branch from 12f92b2 to 12c0976 Compare October 21, 2024 11:25
@goapunk goapunk requested a review from m4ra October 21, 2024 11:25
@goapunk
Copy link
Contributor Author

goapunk commented Oct 22, 2024

@m4ra this should hopefully be ready again

m4ra
m4ra approved these changes Oct 22, 2024
View reviewed changes
Copy link
Contributor

@m4ra m4ra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@m4ra m4ra merged commit ff53e5d into main Oct 22, 2024
2 of 3 checks passed
@m4ra m4ra deleted the jd-2024-09-delete-opin-apps-part2 branch October 22, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m4ra m4ra m4ra approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@goapunk @m4ra