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

Issue 223064: update README for SQLite (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years ago by ced
Modified:
16 years ago
Reviewers:
bch, yangoon1
Visibility:
Public.

Patch Set 1 #

Total comments: 4

Patch Set 2 : Fix typo #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M README View 1 1 chunk +6 lines, -0 lines 1 comment Download

Messages

Total messages: 4
ced
16 years ago (2010-02-26 11:06:34 UTC) #1
yangoon1
http://codereview.appspot.com/223064/diff/1/2 File README (right): http://codereview.appspot.com/223064/diff/1/2#newcode34 README:34: ----------------- Database Backends (RDBMS) ------------------------- http://codereview.appspot.com/223064/diff/1/2#newcode37 README:37: available. Here ...
16 years ago (2010-02-26 12:15:08 UTC) #2
ced
16 years ago (2010-02-26 14:30:06 UTC) #3
yangoon1
16 years ago (2010-02-26 15:03:40 UTC) #4
http://codereview.appspot.com/223064/diff/1002/6
File README (right):

http://codereview.appspot.com/223064/diff/1002/6#newcode56
README:56: * SQL constraints are validated by Tryton instead of database.
instead of database.

Not ok for me, since the DBMS validates. This was the cause for my proposal.
Sign in to reply to this message.

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