Reslug News Frequently Asked Questions #2936

Merged
merged 1 commit into from Jan 4, 2017

Projects

None yet

4 participants

@mgrassotti
Contributor

Trello: https://trello.com/c/YYaiCU7Y/548-slug-change-required

Modify the slug of the document into
"Information-about-the-UK-leaving-the-EU"

+article = NewsArticle.find(688810)
+document = article.document
+
+document.update_attributes(slug: "Information-about-the-UK-leaving-the-EU")
@fofr
fofr Dec 22, 2016 Member

Should it be information not Information?

@fofr
fofr Dec 22, 2016 Member

Zendesk ticket has been updated saying it should be lowercase: https://govuk.zendesk.com/agent/tickets/1480095

@fofr

Use information not Information

@mgrassotti
Contributor

👍

@fofr
fofr approved these changes Jan 3, 2017 View changes
@boffbowsh

The new slug needs to be lowercase

@mgrassotti @boffbowsh mgrassotti Reslug News Frequently Asked Questions
Modify the slug of the document into
"information-about-the-uk-leaving-the-eu"
1003ecf
@fofr
Member
fofr commented Jan 3, 2017 edited

@boffbowsh It is now, isn't it? That's why I approved.
document.update_attributes(slug: "information-about-the-uk-leaving-the-eu")

@boffbowsh
Member

I force-pushed the fix after I commented, once I remembered Matteo was off all week

@gpeng
gpeng approved these changes Jan 4, 2017 View changes
@boffbowsh boffbowsh merged commit 0f99d19 into master Jan 4, 2017

1 check passed

continuous-integration/jenkins/branch This commit looks good
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment