Review Request: Block/Unblock feature in ktp-text-ui module.

[Martin Klapetek]

> On March 30, 2012, 6:17 p.m., Daniele Elmo Domenichelli wrote:
> > Ship It!
> 
> Ahmed Ibrahim wrote:
>     I don't have the rights to commit the code. So could you please commit it.
> 
> Daniele Elmo Domenichelli wrote:
>     Please push it in a personal clone (see http://community.kde.org/Sysadmin/GitKdeOrgManual) and ping on IRC, I will pull your commits from there an push them in the main repository.
>     
>     Also remember to add to the last commit
>     
>     FEATURE: <number of the bug/wish>  (if it is just a bug use BUG: instead)
>     REVIEW: <number of this review>
>     Reviewed-by: Name(s) of the person(s) who reviewed the patch
> 
> David Edmundson wrote:
>     DrDanz - I don't think he can without commit access.
>     Just click "download diff", then patch.
> 
> Daniele Elmo Domenichelli wrote:
>     If he can't commit to a private clone then something is broken with kde git... Anyway I think he can, because hooks should not be enabled on private clones.
>     Otherwise push them to some other public repository (gitorious, github, ...), or send me the patch(es) using "git format-patch origin/master"
>
> 
> Martin Klapetek wrote:
>     If you don't have git account, you can't commit anywhere in the kde git infrastructure, including private scratches. You can simply download the diff up here on the right and do "git apply downloaded_patch.diff", it works perfectly fine.
> 
> Daniele Elmo Domenichelli wrote:
>     That's not fine for me, you lose the author of the patch in git history, if the patch is made of several commits, you lose them.
>     Git is supposed to be used like that.
>     Moreover if he never used git, he has to learn how to use it... by learning how and what to push in a private repository, it is more unlikely that he will screw up the main repository when he get rights to commit there.

You have git commit --author "Name <email>" and it's one patch, the one in this review. We do this all the time... 

And even if he would learn how to use git, which no doubt he should, he still don't have KDE git account and would have to apply for one and git accounts are not given for one (rather simple) patch, so he won't be able to commit it anyway. Not sooner than a month anyway...


- Martin


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


On March 30, 2012, 4:28 p.m., Ahmed Ibrahim wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104407/
> -----------------------------------------------------------
> 
> (Updated March 30, 2012, 4:28 p.m.)
> 
> 
> Review request for Telepathy and Daniele Elmo Domenichelli.
> 
> 
> Description
> -------
> 
> Added Block/Unblock contact button to the KActionCollection of the ktp-text-ui.
> Added also code that will help in displaying blocking/unblocking actions to the user in the ChatView.
> 
> 
> This addresses bug 292402.
>     http://bugs.kde.org/show_bug.cgi?id=292402
> 
> 
> Diffs
> -----
> 
>   app/chat-window.h fd81141 
>   app/chat-window.cpp bd4fc35 
>   app/chatwindow.rc ac5a259 
>   app/main.cpp d76ce04 
>   lib/channel-contact-model.h 41d5d84 
>   lib/channel-contact-model.cpp d731322 
>   lib/chat-widget.h 10de317 
>   lib/chat-widget.cpp 0a0b5db 
> 
> Diff: http://git.reviewboard.kde.org/r/104407/diff/
> 
> 
> Testing
> -------
> 
> Blocked a contact and found that is blocked in another IM client and had other opened tabs and the button was changing according to the user's block state 
> 
> 
> Thanks,
> 
> Ahmed Ibrahim
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120331/69063f1e/attachment.html>