Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29339091: Noissue - Allow retrieving arbitrary info.* properties (Closed)

Created:
March 29, 2016, 9:50 a.m. by Sebastian Noack
Modified:
March 29, 2016, 12:37 p.m.
Reviewers:
kzar
CC:
Thomas Greiner
Visibility:
Public.

Description

Noissue - Allow retrieving arbitrary info.* properties

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -5 lines) Patch
M messageResponder.js View 2 chunks +1 line, -5 lines 0 comments Download

Messages

Total messages: 2
Sebastian Noack
Besides simplifying the message responder logic for app.get, this will also allow to retrieve any ...
March 29, 2016, 9:53 a.m. (2016-03-29 09:53:41 UTC) #1
kzar
March 29, 2016, 11:14 a.m. (2016-03-29 11:14:28 UTC) #2
LGTM

(I'm already using this logic with my work on the old options page. It needs
access to a few extra properties of `info`.)

Powered by Google App Engine
This is Rietveld