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

Remove worldwide priority data #2563

Merged
merged 2 commits into from Apr 29, 2016
Merged

Remove worldwide priority data #2563

merged 2 commits into from Apr 29, 2016

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Apr 21, 2016

  • Delete all join table content for worldwide priorities
    • Edition relationships
    • Document relationships
    • Edition dependencies
  • Deletes the worldwide priorities
  • Deletes data before deleting tables, which will be in follow-up PR

https://trello.com/c/AtYGioxe/321-worldwide-priorities-deleting-the-format-medium

Depends on https://github.gds/gds/router-data/pull/535 being deployed.

Originally written as part of:
https://trello.com/c/UbEYd9y5/328-remove-worldwide-priorities-data-from-whitehall

Old PR: #2247

cc @boffbowsh

elliotcm and others added 2 commits Jun 25, 2015
These have been migrated to policy papers and the
format is being retired.
They’ve already been deleted.
Unpublishing,
].each do |edition_join_model|
join_models = edition_join_model.where(edition_id: edition_ids)
puts "Deleting all #{join_models.count} #{edition_join_model} edition join models"

This comment has been minimized.

@boffbowsh

boffbowsh Apr 21, 2016
Contributor

Should be records, not models.

@boffbowsh
Copy link
Contributor

@boffbowsh boffbowsh commented Apr 21, 2016

Tiny nitpicky comment, otherwise 👍

@fofr fofr changed the title [Do not merge] Remove worldwide priority data Remove worldwide priority data Apr 22, 2016
@fofr fofr merged commit f4ada3c into master Apr 29, 2016
1 check passed
1 check passed
default Build #8166 succeeded on Jenkins
Details
@fofr fofr deleted the remove-worldwide-priority-data branch Apr 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.