nats-io / natscli

The NATS Command Line Interface
Apache License 2.0
510 stars 99 forks source link

Fix columns for mirrors under `Replication Report` #1187

Closed neilalexander closed 1 week ago

neilalexander commented 1 week ago

Previously there were too many columns for the headers, and Active was named incorrectly:

╭──────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│                                                Replication Report                                                │
├──────────────────┬────────┬──────────────┬──────────────────┬────────────────────────┬────────┬───────┬───────┬──┤
│ Stream           │ Kind   │ API Prefix   │ Source Stream    │ Filters and Transforms │ Active │ Lag   │ Error │  │
├──────────────────┼────────┼──────────────┼──────────────────┼────────────────────────┼────────┼───────┼───────┼──┤
│ stream           │ Mirror │ $JS.home.API │ source           │                        │        │ 528ms │ 0     │  │
╰──────────────────┴────────┴──────────────┴──────────────────┴────────────────────────┴────────┴───────┴───────┴──╯

... when they should have been lined up like so, with Seen instead:

╭─────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│                                              Replication Report                                             │
├──────────────────┬────────┬──────────────┬──────────────────┬────────────────────────┬────────┬─────┬───────┤
│ Stream           │ Kind   │ API Prefix   │ Source Stream    │ Filters and Transforms │ Seen   │ Lag │ Error │
├──────────────────┼────────┼──────────────┼──────────────────┼────────────────────────┼────────┼─────┼───────┤
│ stream           │ Mirror │ $JS.home.API │ source           │                        │ 100ms  │ 0   │       │
╰──────────────────┴────────┴──────────────┴──────────────────┴────────────────────────┴────────┴─────┴───────╯

Signed-off-by: Neil Twigg neil@nats.io

ripienaar commented 1 week ago

Thank you!