Add import to jsonify example #700

Merged
merged 1 commit into from Apr 1, 2013

Conversation

3 participants
@SpOOnman
Contributor

SpOOnman commented Mar 25, 2013

I am new to flask and it took me half an hour to figure out why jsonify doesn't work for me. So answer has pointed out solution for me: http://stackoverflow.com/questions/10727366/jsonify-is-not-defined-internal-server-error . I thought that this import line in example can help out newcomers, like me, in a future.

@untitaker

This comment has been minimized.

Show comment Hide comment
@untitaker

untitaker Mar 25, 2013

Member

You might argue that the example is already incomplete because it doesn't contain code to create an app. I'm +0 on this.

Member

untitaker commented Mar 25, 2013

You might argue that the example is already incomplete because it doesn't contain code to create an app. I'm +0 on this.

@SpOOnman

This comment has been minimized.

Show comment Hide comment
@SpOOnman

SpOOnman Mar 25, 2013

Contributor

But you already have an app and add non-working example. Maybe you knew this import already, I didn't since it is not mentioned anywhere.

Contributor

SpOOnman commented Mar 25, 2013

But you already have an app and add non-working example. Maybe you knew this import already, I didn't since it is not mentioned anywhere.

@kennethreitz

This comment has been minimized.

Show comment Hide comment
@kennethreitz

kennethreitz Apr 1, 2013

Contributor

This clarifies things. Thanks!

Contributor

kennethreitz commented Apr 1, 2013

This clarifies things. Thanks!

kennethreitz added a commit that referenced this pull request Apr 1, 2013

Merge pull request #700 from SpOOnman/master
Add import to jsonify example

@kennethreitz kennethreitz merged commit 84958e5 into pallets:master Apr 1, 2013

1 check failed

default The Travis build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment