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