Issue 74064: Adds support to old message creation.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Issue 74064: Adds support to old message creation.

Can't Edit
Can't Publish+Mail
Start Review

Created:
16 years, 2 months ago by Jinhui Du

Modified:
16 years, 2 months ago

Reviewers:
chabotc

CC:
shindig.remailer_gmail.com

Base URL:
http://svn.apache.org/repos/asf/incubator/shindig/trunk/

Visibility:
Public.

Description

1. Uses HTTP put method to update and the post method to create. It only affects MessageHandler. Since all the other handlers(ActivityHandler, InvalidateHandler, PersonHandler, AppDataHandler) either treat the post and the put method as the same or don't support the put method. 2. Adds the logic to support old message creation.

Patch Set 1 #

Created: 16 years, 2 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+15 lines, -7 lines)			Patch
		php/src/social/service/DataRequestHandler.php	View		1 chunk	+2 lines, -2 lines	0 comments	Download
		php/src/social/service/MessagesHandler.php	View		3 chunks	+13 lines, -5 lines	0 comments	Download

Messages

Total messages: 2

Expand All Messages | Collapse All Messages

Jinhui Du

16 years, 2 months ago (2009-06-16 13:47:19 UTC) #2

On 2009/06/16 13:45:02, Jinhui Du wrote:
> 

This change list overwrites the change http://codereview.appspot.com/63230.

Please also have a look at the tiny change
http://codereview.appspot.com/67171/show

Expand All Messages | Collapse All Messages