D10990: Add "Open symlink destination folder" into symlink's context menu
Henrik Fehlauer
noreply at phabricator.kde.org
Wed Mar 14 20:12:05 GMT 2018
rkflx added a comment.
In D10990#225898 <https://phabricator.kde.org/D10990#225898>, @ngraham wrote:
> Yes, I did the code review. Did I miss something obvious?
I don't know, because I did not look at the code. But at least you could point out when "Accepting" what you did and what you didn't check, because you accepted things in the past were you clearly did not look at the code and only did QA (and I'm also not sure how familiar you are with Dolphin's code base, TBH).
REPOSITORY
R318 Dolphin
REVISION DETAIL
https://phabricator.kde.org/D10990
To: rominf, #dolphin, ngraham
Cc: rkflx, ngraham, elvisangelaccio, markg, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180314/0f8a0544/attachment.htm>
More information about the kfm-devel
mailing list