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

Reverts acronym change #229

Merged
merged 2 commits into from
Jan 5, 2022
Merged

Reverts acronym change #229

merged 2 commits into from
Jan 5, 2022

Conversation

1pretz1
Copy link
Contributor

@1pretz1 1pretz1 commented Jan 5, 2022

This code is buggy as it replaces abbreviations in HTML tag attributes,
rather than just text tags. This has knock on consequences of removing
HTML attachments from documents in Whitehall when creating a new
edition.

This commit reverts the change for now with a view to updating the
logic in the near future.

Original commit:
https://github.com/alphagov/govspeak/pull/216/files#diff-acb6db6cd67285bd5380398c9e51ced8940ed7c4110706d038cbb243310e4055R453

Zendesk:
https://govuk.zendesk.com/agent/tickets/4819383

This code is buggy as it replaces abbreviations in HTML tag attributes,
rather than just text tags. This has knock on consequences of removing
HTML attachments from documents in Whitehall when creating a new
edition.

This commit reverts the change for now with a view to updating the
logic in the near future.

Zendesk:
https://govuk.zendesk.com/agent/tickets/4819383
@1pretz1 1pretz1 changed the title wip Reverts acronym change Jan 5, 2022
@1pretz1 1pretz1 merged commit 0198a11 into main Jan 5, 2022
@1pretz1 1pretz1 deleted the fix-bug branch January 5, 2022 14:07
1pretz1 added a commit to alphagov/whitehall that referenced this pull request Jan 5, 2022
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