<div dir="ltr">Gotcha, somehow yesterday I couldn't recall how to create a new revision in phabricator. it's there now.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jun 16, 2019 at 3:33 AM Thomas Baumgart <<a href="mailto:thb@net-bembel.de">thb@net-bembel.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Jeremy,<br>
<br>
On Sonntag, 16. Juni 2019 05:05:32 CEST Jeremy Whiting wrote:<br>
<br>
> Hello,<br>
> <br>
> It's been far too long since I've last made a change to some kde repository<br>
> that I don't maintain (and those that I do also tbh) I came up with a<br>
> quick/small patch to hide the liability accounts that have a zero balance<br>
> on the home screen. It cleans up my home screen quite a bit since I have<br>
> some liability accounts I haven't closed, but that have a zero balance and<br>
> don't typically need to be concerned about. I've pushed the patch to a<br>
> hide-zero-liabilities branch and attached it here. Is phabricator typically<br>
> used for code review lately? or just pushing branches and mentioning it in<br>
> the mailing list here? or something else entirely?<br>
<br>
Phabricator is the way to go (until we move to Gitlab when the general decision has been made). <br>
<br>
The patch looks good in general. I do have a few minor comments and improvements, though. Would it make sense to extent it to also cover asset accounts? But we could also discuss that on Phabricator.<br>
<br>
Thomas<br>
<br>
<br>
-- <br>
<br>
Regards<br>
<br>
Thomas Baumgart<br>
<br>
<a href="https://www.signal.org/" rel="noreferrer" target="_blank">https://www.signal.org/</a>       Signal, the better WhatsApp<br>
-------------------------------------------------------------<br>
God is real... unless declared an integer<br>
-------------------------------------------------------------<br>
</blockquote></div>