From 51fc4b7a43846420a7aac68837ae3019de767ca4 Mon Sep 17 00:00:00 2001 From: Jerzy Spendel Date: Tue, 18 Aug 2015 15:15:12 +0200 Subject: [PATCH 1/3] Tuple in pipeline's documentation should be ended with coma --- docs/pipeline.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/pipeline.rst b/docs/pipeline.rst index 64e65fb60..3606ae5b8 100644 --- a/docs/pipeline.rst +++ b/docs/pipeline.rst @@ -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', ) From 69fa698aee3c66daf3928447d5869711fadd5a61 Mon Sep 17 00:00:00 2001 From: Jerzy Spendel Date: Tue, 18 Aug 2015 15:19:27 +0200 Subject: [PATCH 2/3] Coma at the end of every tuple and dict --- docs/pipeline.rst | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/pipeline.rst b/docs/pipeline.rst index 3606ae5b8..458eeacc4 100644 --- a/docs/pipeline.rst +++ b/docs/pipeline.rst @@ -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 @@ -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', ) @@ -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 @@ -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 From bdd774871a653254ef08f46ea4d0914095ad675a Mon Sep 17 00:00:00 2001 From: Jerzy Spendel Date: Wed, 19 Aug 2015 10:04:59 +0200 Subject: [PATCH 3/3] One more coma in use_cases.rst --- docs/use_cases.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/use_cases.rst b/docs/use_cases.rst index 38c25ba09..bebb60a81 100644 --- a/docs/use_cases.rst +++ b/docs/use_cases.rst @@ -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