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

Issue 184066: Updates to WebFinger wiki (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years, 5 months ago by jcgregorio
Modified:
16 years, 1 month ago
Reviewers:
jpanzer, jcgregorio
CC:
webfinger_googlegroups.com
Base URL:
http://webfinger.googlecode.com/svn/wiki/
Visibility:
Public.

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -2 lines) Patch
A Requirements.wiki View 1 chunk +35 lines, -0 lines 4 comments Download
M WebFingerProtocol.wiki View 2 chunks +4 lines, -2 lines 0 comments Download

Messages

Total messages: 2
jpanzer
Review of Joe's suggested Wiki updates; LGTM. http://codereview.appspot.com/184066/diff/1/2 File Requirements.wiki (right): http://codereview.appspot.com/184066/diff/1/2#newcode5 Requirements.wiki:5: follow requirements, ...
16 years, 5 months ago (2010-01-12 01:05:51 UTC) #1
jcgregorio
16 years, 5 months ago (2010-01-12 14:18:29 UTC) #2
Brad gave me commit access, so I've committed it with all
of these changes.


On 2010/01/12 01:05:51, jpanzer wrote:
> Review of Joe's suggested Wiki updates; LGTM.
> 
> http://codereview.appspot.com/184066/diff/1/2
> File Requirements.wiki (right):
> 
> http://codereview.appspot.com/184066/diff/1/2#newcode5
> Requirements.wiki:5: follow requirements, which will give some insight
> typo: following
> 
> http://codereview.appspot.com/184066/diff/1/2#newcode11
> Requirements.wiki:11: able to use WebFinger. That implies that is must
> typo: it?  (implies that at most a user must be able to create a directory and
> place a file in it to be served?)
> 
> http://codereview.appspot.com/184066/diff/1/2#newcode14
> Requirements.wiki:14: is required, picking a technology that allows packaging
> A bit unclear -- perhaps "needed in some situations"?
> 
> http://codereview.appspot.com/184066/diff/1/2#newcode24
> Requirements.wiki:24: defacement attack could lead users to enter their
> typo: attack,
Sign in to reply to this message.

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