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

Add HMRC webchat to ‘manage your tax credits’ page #794

Merged
merged 2 commits into from
May 20, 2016

Conversation

daibach
Copy link
Contributor

@daibach daibach commented May 19, 2016

Request from HMRC

To be merged and deployed on Friday 20 May 2016.

@boffbowsh
Copy link
Contributor

👍

@issyl0
Copy link
Contributor

issyl0 commented May 19, 2016

This looks like what we did the last time we had to add it to a page, so 👍.

@@ -25,6 +25,7 @@
'/vat-registration/changes-to-your-details',
'/government/organisations/hm-revenue-customs/contact/national-insurance-numbers',
'/lost-national-insurance-number',
'/manage-your-tax-credits',
Copy link
Contributor

Choose a reason for hiding this comment

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

Trailing comma will break in IE7+8. Can it be removed?

Copy link
Contributor

@issyl0 issyl0 May 19, 2016

Choose a reason for hiding this comment

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

We tend to keep trailing commas around to reduce noisy diffs when the next change comes (otherwise you'd have two lines changed because of adding a comma to the previous line) but yeah, if it breaks something... 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

JavaScript is the exception to this rule. Trailing commas break older IEs and are hard to track down.

It would probably break IE
@boffbowsh
Copy link
Contributor

FTFY

@daibach daibach merged commit 9662fba into master May 20, 2016
@boffbowsh boffbowsh deleted the hmrc-webchat-addition-20160519 branch May 20, 2016 08:28
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.

5 participants