Patch 16004: Issue 117: Invite, confirmation and lost password emails contain incorrect mobile links

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file 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
<Enter> :	respond to / edit current comment

	Patch
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

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

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

Patch 16004: Issue 117: Invite, confirmation and lost password emails contain incorrect mobile links

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

Created:
6 months ago by lemonad

Modified:
2 months, 3 weeks ago

Reviewers:
termie, adewale

CC:

SVN Base:
http://jaikuengine.googlecode.com/svn/trunk/

Visibility:
Public.

Description

Not certain about the change improving things on Jaiku.com as the settings.py
file is not public but based on the other code, it should work.

Patch by: Jonas Nockert

Patch Set 1

Created: 6 months ago

Download raw patch set

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats			Patch
	M	common/mail.py	View		3 chunks	31 lines	0 comments	Download

Messages

Total messages: 1

Expand All Messages | Collapse All Messages

The hosted domain for Jaiku is jaiku.com not www.jaiku.com so this will fix it.

However this looks like it won't work for any JaikuEngine instance that's served
from $FOO.appspot.com as the url will get changed to m.appspot.com

It also won't work for users who use the command: python manage.py config
--write-to-file to set up their JaikuEngine instance as their HOSTED_DOMAIN
property won't exist.

Expand All Messages | Collapse All Messages