Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(103)

Issue 9229046: TRIVIAL: Fixes a template variable name

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 10 months ago by j.c.sackett
Modified:
10 years, 10 months ago
Reviewers:
mp+163993
Visibility:
Public.

Description

TRIVIAL: Fixes a template variable name A template references source_link, but the tplData has sourceLink; this just updates the template to use sourceLink. https://code.launchpad.net/~jcsackett/juju-gui/source-link-should-have-url/+merge/163993 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : TRIVIAL: Fixes a template variable name #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M app/subapps/browser/templates/browser_charm.handlebars View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
j.c.sackett
Please take a look.
10 years, 10 months ago (2013-05-15 16:22:26 UTC) #1
j.c.sackett
LGTM Self reviewing trivial change.
10 years, 10 months ago (2013-05-15 16:23:56 UTC) #2
j.c.sackett
10 years, 10 months ago (2013-05-15 16:30:18 UTC) #3
*** Submitted:

TRIVIAL: Fixes a template variable name

A template references source_link, but the tplData has sourceLink; this just
updates the template to use sourceLink.

R=
CC=
https://codereview.appspot.com/9229046
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b