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

Issue 6887049: Renaming _MergeFromKey, making it public, and making it directly consume a key. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 8 months ago by dhermes
Modified:
13 years, 7 months ago
Reviewers:
kdeus
CC:
fredsa.google
Visibility:
Public.

Description

Renaming _MergeFromKey, making it public, and making it directly consume a key.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -25 lines) Patch
M endpoints_proto_datastore/ndb/model.py View 3 chunks +16 lines, -17 lines 0 comments Download
M examples/custom_alias_properties/main.py View 1 chunk +1 line, -3 lines 0 comments Download
M examples/keys_with_ancestors/main.py View 2 chunks +3 lines, -5 lines 0 comments Download

Messages

Total messages: 4
dhermes
13 years, 8 months ago (2012-12-06 02:10:49 UTC) #1
kdeus
Much cleaner. Looks good.
13 years, 8 months ago (2012-12-06 02:36:28 UTC) #2
dhermes
Added in http://code.google.com/p/endpoints-proto-datastore/source/detail?r=c4463c2c6f3ca33eb4b25752c39785ad09d0e539
13 years, 8 months ago (2012-12-06 17:15:52 UTC) #3
dhermes
13 years, 7 months ago (2013-01-18 06:57:58 UTC) #4
Message was sent while issue was closed.
The commit this was added in has since moved to

http://code.google.com/p/endpoints-proto-datastore/source/detail?r=078e894310...

since the repository migrated from mercurial to git.
Sign in to reply to this message.

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