[bisq-network/bisq] Add missing filtering on lists (PR #5963)

chimp1984 notifications at github.com
Fri Jan 21 22:33:51 CET 2022


> I think we should unify it (personally I think B approach is better). What do you think?

Yes agree. Don't remember the reasons why that was in the ViewModel, maybe because it accessed other ViewDataModel code or as it was already used from earlier code. But I also dislike that.

Rest looks good to me. Not sure if I find time for a full review... If @ripcurlx does not find time ping me again. 

Thanks.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/5963#issuecomment-1018880254
You are receiving this because you are subscribed to this thread.

Message ID: <bisq-network/bisq/pull/5963/c1018880254 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20220121/048e037f/attachment-0001.htm>


More information about the bisq-github mailing list