[bisq-network/bisq] Refactor CLI output formatting code & add trade formatter (#4703)

Stan notifications at github.com
Fri Oct 30 14:56:14 UTC 2020


@ghubstan commented on this pull request.



> +
+import com.google.common.annotations.VisibleForTesting;
+
+import java.util.function.Supplier;
+
+import static bisq.cli.ColumnHeaderConstants.*;
+import static bisq.cli.CurrencyFormat.formatOfferPrice;
+import static bisq.cli.CurrencyFormat.formatSatoshis;
+import static com.google.common.base.Strings.padEnd;
+
+ at VisibleForTesting
+public class TradeFormat {
+
+    @VisibleForTesting
+    public static String format(TradeInfo tradeInfo) {
+        // Some column values might be longer than header, so we need to calculated them.

Resolved in PR https://github.com/bisq-network/bisq/pull/4731, commit https://github.com/bisq-network/bisq/pull/4731/commits/3cd3bf0e3ed3aed626fe38ef1b1da29cbc761c61.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4703#discussion_r515159321
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20201030/802b96e2/attachment.html>


More information about the bisq-github mailing list