Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix to issue reported by certik #11

Closed
wants to merge 2 commits into from
Closed

fix to issue reported by certik #11

wants to merge 2 commits into from

Conversation

sharpred
Copy link

the issue "Stacktrace at /todo/" reported by certik can be fixed by pulling my copies of email.html and todo.html

line 50 of todo.html was changed from {{ url_for('add_todo') }} to {{ url_for('views.add_todo') }}

the same issue in email.html can be fixed by changing line 61 from{{ url_for('email_someone') }} to {{ url_for('views.email_someone') }}

@sharpred
Copy link
Author

actually - you only need to apply the second commit - first was my secret key and project id changes :-(

zachwill added a commit that referenced this pull request Jan 21, 2013
@zachwill
Copy link
Owner

Thanks, @sharpred! Cherry picked the template updates.

@zachwill zachwill closed this Jan 21, 2013
@sharpred
Copy link
Author

NP's - thanks for sharing this repo. It's really good for a GAE/Python newb like me

@zachwill
Copy link
Owner

Glad it's helping out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants