Skip to content

Commit

Permalink
Merge pull request #1468 from sul-dlss/t1457-bw-fixes
Browse files Browse the repository at this point in the history
Correct issues with Boundwith App
  • Loading branch information
jermnelson authored Dec 4, 2024
2 parents 641a08d + 2fa5d01 commit c32414c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion libsys_airflow/dags/new_boundwiths.py
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ def new_admin_notes(**kwargs):

admin_note = generate_admin_note()

start >> [rows, admin_note]
start >> rows >> admin_note

bw_records = add_bw_record.expand(row=rows)

Expand Down
2 changes: 1 addition & 1 deletion libsys_airflow/plugins/boundwith/boundwith_view.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ def run_bw_creation(self):
raw_csv = request.files["upload-boundwith"]
email_addr = request.form.get("user-email")
bw_df = pd.read_csv(raw_csv)
if ["parts_holdings_hrid", "principle_barcode"] != list(bw_df.columns):
if ["part_holdings_hrid", "principle_barcode"] != list(bw_df.columns):
flash(f"Invalid columns: {list(bw_df.columns)} for CSV file")
rendered_page = self.render_template("boundwith/index.html")
elif len(bw_df) > 1_000:
Expand Down

0 comments on commit c32414c

Please sign in to comment.