JaikuEngine Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 years, 3 months ago by lemonad
Modified:
4 years, 7 months 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

Patch Set 2 : Re-tested with latest revision of Jaikuengine

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

Messages

Total messages: 2
adewale
The hosted domain for Jaiku is jaiku.com not www.jaiku.com so this will fix it. However ...
5 years, 2 months ago
lemonad
4 years, 7 months ago
On 2009/06/08 22:15:47, adewale wrote:
> The hosted domain for Jaiku is http://jaiku.com not http://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 http://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.

I'll investigate and upload a new patch later!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Patches | This issue
This is Rietveld r