search

alphagov / specialist-publisher

Publishes specialist documents on GOV.UK
https://docs.publishing.service.gov.uk/apps/specialist-publisher.html
MIT License
10 stars 7 forks source link

[CLOSED] Correct the MAIB organisation slug #471

Closed benilovj closed 8 years ago

benilovj commented 8 years ago

Issue by jamiecobbett Thursday 16 April 2015 at 13:07 GMT Originally opened as https://github.com/alphagov/specialist-publisher/pull/471

This was added in 2d0a48c4d8b5ae9524ad5b40c25d28d4678ed1c4 and seems to have just been a typo.

I don't think just changing this is enough to fix the existing bad metadata on search results - presumably all of the reports need to be resent to Panopticon/Rummager somehow?

jamiecobbett included the following code: https://github.com/alphagov/specialist-publisher/pull/471/commits

benilovj commented 8 years ago

Comment by tommyp Thursday 16 April 2015 at 13:25 GMT

MAIB's slug and name were mismatched at the time of creating the MAIB finder so this was correct at that time. I think in order to correct it they all need to be republished.

benilovj commented 8 years ago

Comment by jamiecobbett Thursday 16 April 2015 at 14:58 GMT

After testing in development (and adding support for them in the last commit), I will run:

govuk_setenv specialist-publisher bundle exec ruby bin/republish_documents maib_report

To reindex the documents. Hat tip to @evilstreak for being awesome.

benilovj commented 8 years ago

Comment by evilstreak Thursday 16 April 2015 at 16:53 GMT

:+1: As a follow up, we'll move organisation slugs into constants to make this kind of mistake impossible and make managing slug changes a bit less painful