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

Update legal overview page copy #2527

Merged
merged 2 commits into from Dec 13, 2017

Conversation

anthonydillon
Copy link
Contributor

Done

Update legal overview page copy to match the copy doc.

QA

  • Check out this feature branch
  • Run the site using the command ./run serve
  • View the site locally in your web browser at: http://0.0.0.0:8001/legal
  • Check that the copy matches the copy in the copy doc

Issue / Card

Fixes #2520

@webteam-app
Copy link

webteam-app commented Dec 11, 2017

Starting demo at: http://www.ubuntu.com-pr-2527.run.demo.haus/legal

Copy link
Contributor

@Lukewh Lukewh left a comment

Choose a reason for hiding this comment

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

Just one question about the title

@@ -9,7 +9,7 @@
<div class="row">
<div class="col-8">
<h1>Ubuntu legal terms and&nbsp;policies</h1>
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if relevant, but this title is different to the copydoc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed thanks, good spot.

@anthonydillon anthonydillon merged commit 4efe021 into canonical:master Dec 13, 2017
@anthonydillon anthonydillon deleted the update-legal-copy branch December 13, 2017 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants