Merge lp:~devcamcar/django-nova/lp711010 into lp:django-nova

Proposed by Devin Carlen
Status: Merged
Approved by: Devin Carlen
Approved revision: 18
Merged at revision: 15
Proposed branch: lp:~devcamcar/django-nova/lp711010
Merge into: lp:django-nova
Diff against target: 62 lines (+42/-1)
2 files modified
src/django_nova/templates/django_nova/credentials/expired.html (+5/-1)
src/django_nova/templatetags/django_nova_tags.py (+37/-0)
To merge this branch: bzr merge lp:~devcamcar/django-nova/lp711010
Reviewer Review Type Date Requested Status
Devin Carlen Pending
Ken Pepple Pending
Review via email: mp+48228@code.launchpad.net

This proposal supersedes a proposal from 2011-02-01.

Description of the change

Implemented site branding template tags to reduce the need to override templates

To post a comment you must log in.
Revision history for this message
Ken Pepple (ken-pepple) wrote : Posted in a previous version of this proposal

i'm not sure how to expire credentials, but the rest of the site branding (minus /admin which i reported in bug report 711024) looks good.

review: Approve
Revision history for this message
Devin Carlen (devcamcar) wrote :

Simplified the implementation and fixed the admin page.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/django_nova/templates/django_nova/credentials/expired.html'
--- src/django_nova/templates/django_nova/credentials/expired.html 2011-01-31 20:08:02 +0000
+++ src/django_nova/templates/django_nova/credentials/expired.html 2011-02-01 19:22:18 +0000
@@ -1,3 +1,4 @@
1{% load django_nova_tags %}
1<!DOCTYPE html>2<!DOCTYPE html>
2<html lang="en" xml:lang="en">3<html lang="en" xml:lang="en">
3 <head>4 <head>
@@ -7,7 +8,10 @@
7 <body>8 <body>
8 <center>9 <center>
9 <h1>The link you clicked has expired.</h1>10 <h1>The link you clicked has expired.</h1>
10 <p style="width:460px;">This credentials download link you have reached is either invalid or has expired. Each link is only good for one use. If you need to download your credentials again, please contact the support team.</p>11 <p style="width:460px;">This credentials download link you have reached
12 is either invalid or has expired. Each link is only good for one use. If
13 you need to download your credentials again, please contact the
14 {% site_branding %} support team.</p>
11 </center>15 </center>
12 </body>16 </body>
13</html>17</html>
1418
=== added file 'src/django_nova/templatetags/django_nova_tags.py'
--- src/django_nova/templatetags/django_nova_tags.py 1970-01-01 00:00:00 +0000
+++ src/django_nova/templatetags/django_nova_tags.py 2011-02-01 19:22:18 +0000
@@ -0,0 +1,37 @@
1# vim: tabstop=4 shiftwidth=4 softtabstop=4
2
3# Copyright 2010 United States Government as represented by the
4# Administrator of the National Aeronautics and Space Administration.
5# All Rights Reserved.
6#
7# Licensed under the Apache License, Version 2.0 (the "License"); you may
8# not use this file except in compliance with the License. You may obtain
9# a copy of the License at
10#
11# http://www.apache.org/licenses/LICENSE-2.0
12#
13# Unless required by applicable law or agreed to in writing, software
14# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
15# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
16# License for the specific language governing permissions and limitations
17# under the License.
18
19"""
20Template tags for working with django_nova.
21"""
22
23from django import template
24from django.conf import settings
25
26
27register = template.Library()
28
29
30class SiteBrandingNode(template.Node):
31 def render(self, context):
32 return settings.SITE_BRANDING
33
34@register.tag
35def site_branding(parser, token):
36 return SiteBrandingNode()
37

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: