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

Roll back https://github.com/alphagov/govuk-design-system/pull/784 #809

Merged
merged 1 commit into from
Feb 27, 2019

Conversation

hannalaakso
Copy link
Member

@hannalaakso hannalaakso commented Feb 27, 2019

We seem to have introduced a bug in #784 that produces invalid IDs for example markup.

Rolling the PR changes back as an initial fix.

We seem to have introduced a bug that produces invalid IDs for
example markup
@govuk-design-system-ci
Copy link
Collaborator

You can preview this change here:

Built with commit dc6c2c4

https://deploy-preview-809--govuk-design-system-preview.netlify.com

Copy link
Contributor

@36degrees 36degrees left a comment

Choose a reason for hiding this comment

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

👍

@hannalaakso hannalaakso merged commit d3c7074 into master Feb 27, 2019
@hannalaakso hannalaakso deleted the fix-broken-example-ids branch February 27, 2019 11:43
hannalaakso added a commit that referenced this pull request Mar 1, 2019
This reverts a hotfix #809
that removed the changes in favour of a proper fix for
#620

Generating example IDs from example titles can be brittle if there
are unexpected characters in titles. Use the IDs and
fall back to default ones and uses the solution from #620 to fix
duplicate IDs
hannalaakso added a commit that referenced this pull request Mar 4, 2019
This reverts a hotfix #809
that removed the changes in favour of a proper fix for
#620

Generating example IDs from example titles can be brittle if there
are unexpected characters in titles. Use slugger for more robust converting of
exampleTitle to exampleId. This also uses the solution from #620 to fix
duplicate IDs by adding an extra parameter "titleSuffix" to examples if
necessary.

Co-authored-by: Nick Colley <nick.colley@digital.cabinet-office.gov.uk>
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.

3 participants