feat(domain-list): Improved the output of the domain list, use a tabl…

…e to format it nicely #14 Merge pull request #14 from entrostat/feature/domain-list-table
entrostat · Mar 12, 2022 · 62d2db3 · 62d2db3
2 parents a517390 + 0b850e8
commit 62d2db3
Showing 1 changed file with 26 additions and 3 deletions.
diff --git a/src/commands/domain/list.ts b/src/commands/domain/list.ts
@@ -1,4 +1,5 @@
 import { BaseCommand } from '../../shared/base.command';
+import { CliUx } from '@oclif/core';
 
 export default class DomainList extends BaseCommand {
     static description = 'Lists the existing domains that have been registered';
@@ -11,9 +12,31 @@ export default class DomainList extends BaseCommand {
 
     public async run(): Promise<void> {
         const config = await this.getConfig();
-        config.domains.forEach((domain) => {
-            this.log(`127.0.0.1:${domain.port} ---> https://${domain.domain}`);
-        });
+
+        CliUx.ux.table(
+            config.domains
+                .map((d) => ({
+                    domain: `https://${d.domain}`,
+                    port: d.port.toString(),
+                }))
+                .sort((a, b) => {
+                    if (a === b) {
+                        return 0;
+                    }
+                    return a < b ? -1 : 1;
+                }),
+            {
+                domain: {
+                    header: 'Domain',
+                },
+                port: {
+                    header: 'Port',
+                },
+            },
+            {
+                printLine: this.log.bind(this),
+            },
+        );
 
         if (config.domains.length === 0) {
             this.log('No domains registered...');