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

Migrate more calls for evidence #8260

Merged
merged 2 commits into from
Sep 15, 2023
Merged

Conversation

ollietreend
Copy link
Contributor

@ollietreend ollietreend commented Sep 15, 2023

Department for Transport have asked us to migrate 16 more consultations to the new call for evidence type. These were missed from the initial list of documents to migrate.

The request was made to us in Zendesk ticket #5478771.

This CSV contains all 16 documents to be migrated. However 4 of them are in a draft state so will not migrate successfully. I've asked DfT to delete or publish the drafts so the documnets can be migrated.

In the meantime, the 12 published documents will migrate correctly. And we can re-run this migration to catch the final 4 once they're in the correct state.

Migrate these documents by running:

rake migrate_calls_for_evidence:migrate MIGRATION_CSV_PATH="db/data_migration/20230915000000_migrate_calls_for_evidence.csv"

Verify the migration by running:

rake migrate_calls_for_evidence:verify MIGRATION_CSV_PATH="db/data_migration/20230915000000_migrate_calls_for_evidence.csv"

Trello: https://trello.com/c/c31bnJbp/1727-migrate-calls-for-evidence-that-were-missed


⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Follow these steps if you are doing a Rails upgrade.

Update the Call for Evidence migration task so it gracefully skips
documents that've already been migrated.

This makes the Rake task idempotent, so migration CSVs can be re-run
without displaying errors for documents that have already been migrated.
Department for Transport have asked us to migrate 16 more consultations
to the new call for evidence type. These were missed from the initial
list of documents to migrate.

The request was made to us in Zendesk ticket [#5478771][1].

This CSV contains all 16 documents to be migrated. However 4 of them are
in a draft state so will not migrate successfully. I've asked DfT to
delete or publish the drafts so the documnets can be migrated.

In the meantime, the 12 published documents will migrate correctly. And
we can re-run this migration to catch the final 4 once they're in the
correct state.

Migrate these documents by running:

```
rake migrate_calls_for_evidence:migrate MIGRATION_CSV_PATH="20230915000000_migrate_calls_for_evidence.csv"
```

[1]: https://govuk.zendesk.com/agent/tickets/5478771
Copy link
Contributor

@ryanb-gds ryanb-gds left a comment

Choose a reason for hiding this comment

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

👍

@ollietreend ollietreend merged commit 0e050ae into main Sep 15, 2023
9 checks passed
@ollietreend ollietreend deleted the migrate-more-calls-for-evidence branch September 15, 2023 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants