Skip to content
This repository has been archived by the owner on Jan 31, 2018. It is now read-only.

[bug 1055785] Codify data retention policy #331

Merged
merged 2 commits into from
Aug 20, 2014
Merged

[bug 1055785] Codify data retention policy #331

merged 2 commits into from
Aug 20, 2014

Conversation

willkg
Copy link
Member

@willkg willkg commented Aug 20, 2014

This adds the data retention policy to the About page.

It also nixes the mobile template for the About page which wasn't
very functional.

It also adds text and links which are already in the footer, but this
surfaces them so that the About page is "complete" in terms of the
site's policies.

r?

This adds the data retention policy to the About page.

It also nixes the mobile template for the About page which wasn't
very functional.

It also adds text and links which are already in the footer, but this
surfaces them so that the About page is "complete" in terms of the
site's policies.
@@ -6,7 +6,79 @@
<div class="col"></div>
<div class="col wide">
<div class="block">
{% include "includes/about_text.html" %}
<h2>{{ _('Hello!') }}</h2>
Copy link
Contributor

Choose a reason for hiding this comment

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

hi

@rlr
Copy link
Contributor

rlr commented Aug 20, 2014

The language is all Firefox specific. But I see you did't write it.

@rlr
Copy link
Contributor

rlr commented Aug 20, 2014

lgtm r+

@willkg
Copy link
Member Author

willkg commented Aug 20, 2014

Oops. I'll fix this to be product agnostic while I'm changing all the things. Good catch!

willkg added a commit that referenced this pull request Aug 20, 2014
[bug 1055785] Codify data retention policy
@willkg willkg merged commit 8f08ab1 into mozilla:master Aug 20, 2014
@willkg
Copy link
Member Author

willkg commented Aug 20, 2014

Landed in:

91396a8 Generalize About page text so it works for all products
f9ba829 [bug 1055785] Codify data retention policy

@willkg willkg deleted the 1055785-data-retention branch August 21, 2014 01:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants