We are going to use this module in a commerce site.
Test it with place order and checkout, results look great!
However, the email field for this user is empty; when we edit user profile, and input email, save it. there will be error message:
Your current password is missing or incorrect; it's required to change the E-mail address.
So after a qq login user place an order and checkout successfully, no notify email will be sent to this user. On the other hand, almost all the b2b sites are supposed to send an email to the user after he/she has successfully place an order.
I think this is reasonable feature request and thanks for your attention.
Comment | File | Size | Author |
---|---|---|---|
#10 | allow_user_to_set_email-1948808-10.patch | 4.23 KB | edxxu |
Comments
Comment #1
alfababy CreditAttribution: alfababy commentedHi zterry95,
Thanks for your feature request. Looks I can't implement your request now. Because:
1) I can't get user's email address.
2) I can't get user's password.
So I only can give a random password, and no email.
But I can give you a suggestion.
1) You can create a custom form to change email address.
2) Validate the user whether login from third party and email is empty.
3) Use user_save() to save it.
I will think how to set email when user are loginning. And required the email.
Comment #2
miloyz CreditAttribution: miloyz commentedHi Guys,
I don't think implement our own firm is the right way to do. Is it possible to take out the password input field in profile edit form by detecting the user type (QQ user for example)?
I suggest we can look into how other advanced module handle this feature, like facebook or twitter integration module, do they allow use access profile editing page?
Please try not to reinvent the wheel. -_^
Comment #3
DYdave CreditAttribution: DYdave commentedThanks a lot guys for following up on this.
This is a feature that we need badly, since currently the module is not really exploitable with E-Commerce websites for which the email address is particularly important.
Could you perhaps try investigating further how other modules do it?
A few posts I could find:
This doesn't isn't necessarily very easy or straight forward, so I guess there should probably be more discussions and investigations.
@alfababy:
Concretelly speaking, I think we should try to really test all these other third party authentication modules and see if/how they could achieve this feature.
Any further comments, feedback, questions, issues, objections, recommendations would surely be highly appreciated.
Thanks very much to all in advance.
Comment #4
DYdave CreditAttribution: DYdave commentedAny update on this issue?
It would certainly be great to be able to capture more information from users, especially the email address which is really a critical blocker if we would like to use this module on E-Com production sites......
Bumping this to Major.
Any further comments, feedback, questions, issues, objections, recommendations would surely be highly appreciated.
Thanks very much to all in advance.
Comment #5
alfababy CreditAttribution: alfababy commentedLooks this is a major problem now.
I will give a good solution on this soon.
Let me research some similar modules first.
Thanks.
Comment #6
skyredwangA lot of OAuth providers don't provide User email data, for example Twitter and Weibo.
This is an example response from Weibo User request:
For this module, after a successful user login, the callback URL should be a new page that requires user entering email or additional fields required.
Comment #7
edxxu CreditAttribution: edxxu at INsReady commentedAs @skyredwang mentioned, not all OAuth providers provide email data, so i write this patch to allow user set email after login via CSNA link.
Please review this patch, thx.
Comment #8
edxxu CreditAttribution: edxxu at INsReady commentedSorry, the patch in #7 is wrong, here is the patch.
Comment #9
skyredwangNeed a newline here
Comment #10
edxxu CreditAttribution: edxxu at INsReady commentedHere is the new patch, please review.
Comment #11
skyredwangComment #12
rogical CreditAttribution: rogical commentedcommitted, thanks for all.
Comment #14
rogical CreditAttribution: rogical commented