Review Request: Add an option to block an incoming contact

[Rohan Garg]

> On Oct. 23, 2012, 4:22 p.m., David Edmundson wrote:
> > From a code POV, it's good (except for the _massive_ bug that you copied and pasted.)
> > 
> > From a usability request, why would you deny if you didn't want to block as well? It seems having 3 options is a bit redundant IRL, and I would propose merging with deny.
> 
> Rohan Garg wrote:
>     I'd say that 'Deny' would mean "I'm not sure about this person, how about I ask if this really is him before accepting" while 'Block' would mean "Spammer/Stalker!!! Don't ever prompt me again!"
> 
> David Edmundson wrote:
>     And how would you ask about this person?
>     
>     They're not in your roster (so you can't ask them). In _some_ protocols they can't send you messages, in some they can. 
>     In the protocols where they can send you messages when you clicked deny, they could send you messages without asking for your presence anyway so they're not going to be requesting presence subscription.
>
> 
> Rohan Garg wrote:
>     Real life? EMail? IRC?

And as for the protocol issues, I am not sure how to solve that tbh. Sounds like a telepathy issue, but might as well be an issue with the spec itself ( Don't know for certain because I have no idea )

So the only solution that I could think of was to block the contact.


- Rohan


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


On Oct. 23, 2012, 4:30 p.m., Rohan Garg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107012/
> -----------------------------------------------------------
> 
> (Updated Oct. 23, 2012, 4:30 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Add an option to block an incoming contact. This can be used to prevent a notification to be shown when being spammed with new contact requests.
> 
> 
> This addresses bugs 297994 and 308809.
>     http://bugs.kde.org/show_bug.cgi?id=297994
>     http://bugs.kde.org/show_bug.cgi?id=308809
> 
> 
> Diffs
> -----
> 
>   contact-request-handler.h 43456ee 
>   contact-request-handler.cpp ea24885 
> 
> Diff: http://git.reviewboard.kde.org/r/107012/diff/
> 
> 
> Testing
> -------
> 
> Works fine. Once blocked, notification does not reappear. User can also accept a blocked request after unblocking the contact and resending contact authorization.
> 
> 
> Thanks,
> 
> Rohan Garg
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20121023/6f6659fe/attachment.html>