From 86c9065a440d1aacb3004187694a3b0bc4cbd0c8 Mon Sep 17 00:00:00 2001
From: Kenneth Kehl <@kkehl@flexion.us>
Date: Tue, 23 Jul 2024 07:47:35 -0700
Subject: [PATCH] use click.echo

---
 app/commands.py | 6 ++----
 docs/all.md     | 2 +-
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/app/commands.py b/app/commands.py
index c10327108..cca20d626 100644
--- a/app/commands.py
+++ b/app/commands.py
@@ -590,7 +590,7 @@ def process_row_from_job(job_id, job_row_number):
 
 
 @notify_command(name="download-csv-file-by-name")
-@click.option("-f", "--csv_filename", required=True, help="csv file name")
+@click.argument("csv_filename")
 def download_csv_file_by_name(csv_filename):
 
     bucket_name = current_app.config["CSV_UPLOAD_BUCKET"]["bucket"]
@@ -598,9 +598,7 @@ def download_csv_file_by_name(csv_filename):
     secret = current_app.config["CSV_UPLOAD_BUCKET"]["secret_access_key"]
     region = current_app.config["CSV_UPLOAD_BUCKET"]["region"]
     content = s3.get_s3_file(bucket_name, csv_filename, access_key, secret, region)
-    # We want it all to get logged in one line
-    content = content.replace("\n", "|")
-    print(content)
+    click.echo(content)
 
 
 @notify_command(name="populate-annual-billing-with-the-previous-years-allowance")
diff --git a/docs/all.md b/docs/all.md
index d98f0d336..3b1c999d2 100644
--- a/docs/all.md
+++ b/docs/all.md
@@ -1349,7 +1349,7 @@ cf run-task notify-api-production --command "flask command download-csv-file-by-
 locally, just do:
 
 ```
-poetry run flask command download-csv-file-by-name -f <file location in admin logs>
+poetry run flask command download-csv-file-by-name <file location in admin logs>
 ```
 
 ## Debug steps