Review Request: Make creating yahoo! account work with entered @yahoo.com

[Martin Klapetek]

> On Nov. 28, 2011, 5:04 p.m., David Edmundson wrote:
> > Discussing alternatives, you could also have just done:
> > 
> > [                   ]@yahoo.com
> > 
> > and simply placed a label to the right on the text field. It's not a scary warning and is less confusing than changing people's inputs (IMHO)
> 
> Martin Klapetek wrote:
>     Also possible. Though it changes people input only to what they have to enter anyway. The fact that it changes also displayname is another thing (and should be finally solved as well).
> 
> Dario Freddi wrote:
>     However, quick question. How does that deal with, say, @yahoo.it, and in general any other i18n'ed yahoo website?

I just checked with @yahoo.de - I can log in using my @yahoo.com credentials for @yahoo.de - martin.klapetek at yahoo.de works, although I've registered only @yahoo.com. So I assume the part after '@' is not really important.


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103281/#review8572
-----------------------------------------------------------


On Nov. 28, 2011, 1:01 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103281/
> -----------------------------------------------------------
> 
> (Updated Nov. 28, 2011, 1:01 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Haze requires yahoo! account to be without the @yahoo.com part. This patch removes that part if user adds it. It's imho better to do this than to put a big scary WARNING! above the line edit. Our software appears smart.
> 
> 
> This addresses bug 287190.
>     http://bugs.kde.org/show_bug.cgi?id=287190
> 
> 
> Diffs
> -----
> 
>   plugins/haze/yahoo-main-options-widget.h 592e582 
>   plugins/haze/yahoo-main-options-widget.cpp 25d163e 
> 
> Diff: http://git.reviewboard.kde.org/r/103281/diff/diff
> 
> 
> Testing
> -------
> 
> Successfully added a yahoo! account and connected.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20111129/4287b0ce/attachment.html>