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

Issue 29352795: Issue 4377 - Add Configurable Form to Email Service (Episode 2) (Closed)

Created:
Sept. 13, 2016, 8:58 a.m. by Vasily Kuznetsov
Modified:
Sept. 13, 2016, 10:32 a.m.
Reviewers:
Sebastian Noack
CC:
saroyanm, mathias, Philip Hill, Jon Sonesen
Visibility:
Public.

Description

Issue 4377 - Add Configurable Form to Email Service (Episode 2) Repository: hg.adblockplus.org/sitescripts Base revision: 97b22a6a68a0 This is a continuation of https://codereview.adblockplus.org/29352643/ with original changes by Jon.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+236 lines, -0 lines) Patch
M .sitescripts.example View 2 chunks +11 lines, -0 lines 0 comments Download
A sitescripts/formmail/test/template/test.mail View 1 chunk +10 lines, -0 lines 0 comments Download
A sitescripts/formmail/test/test_formmail2.py View 1 chunk +110 lines, -0 lines 0 comments Download
A sitescripts/formmail/web/formmail2.py View 1 chunk +105 lines, -0 lines 0 comments Download

Messages

Total messages: 2
Vasily Kuznetsov
Since Jon is on vacation I'm taking over the #4377 work. I addressed the last ...
Sept. 13, 2016, 9:03 a.m. (2016-09-13 09:03:32 UTC) #1
Sebastian Noack
Sept. 13, 2016, 10:26 a.m. (2016-09-13 10:26:25 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld