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

Tuple in pipeline's documentation should be ended with coma #712

Merged
merged 3 commits into from
Oct 5, 2015
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions docs/pipeline.rst
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ The default pipeline is composed by::
'social.pipeline.social_auth.load_extra_data',

# Update the user record with any changed info from the auth service.
'social.pipeline.user.user_details'
'social.pipeline.user.user_details',
)


Expand All @@ -86,7 +86,7 @@ ones would look like this::
'social.pipeline.social_auth.social_user',
'social.pipeline.social_auth.associate_user',
'social.pipeline.social_auth.load_extra_data',
'social.pipeline.user.user_details'
'social.pipeline.user.user_details',
)

Note that this assumes the user is already authenticated, and thus the ``user`` key
Expand Down Expand Up @@ -144,7 +144,7 @@ In order to override the disconnection pipeline, just define the setting::
'social.pipeline.disconnect.revoke_tokens',

# Removes the social associations.
'social.pipeline.disconnect.disconnect'
'social.pipeline.disconnect.disconnect',
)


Expand Down Expand Up @@ -303,7 +303,7 @@ returned by the provider (``Facebook`` in this example). The usual Facebook
'updated_time': '2014-01-14T15:58:35+0000',
'link': 'https://www.facebook.com/foobar',
'timezone': -3,
'id': '100000126636010'
'id': '100000126636010',
}

Let's say we are interested in storing the user profile link, the gender and
Expand Down Expand Up @@ -333,7 +333,7 @@ the pipeline, since it needs the user instance, it needs to be put after
'path.to.save_profile', # <--- set the path to the function
'social.pipeline.social_auth.associate_user',
'social.pipeline.social_auth.load_extra_data',
'social.pipeline.user.user_details'
'social.pipeline.user.user_details',
)

If the return value of the function is a ``dict``, the values will be merged
Expand Down
2 changes: 1 addition & 1 deletion docs/use_cases.rst
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ function, like this::
'social.pipeline.user.create_user',
'social.pipeline.social_auth.associate_user',
'social.pipeline.social_auth.load_extra_data',
'social.pipeline.user.user_details'
'social.pipeline.user.user_details',
)

This feature is disabled by default because it's not 100% secure to automate
Expand Down