Skip to content

Fix up some links that were still inaccessible.

Uman Shahzad requested to merge uman/alternate-link-transferwise into master

@antoviaque I checked and noticed TransferWise bulk payment CSV links in the billing admin were inaccessible. Indeed, it looks like we may have either missed or accidentally added back the wrong key for getting the link.

See https://github.com/open-craft/accounting/pull/12 which had originally fixed this.

Testing instructions:

  1. Go to production to see the existing TransferWise bulk payments and notice you cannot view the CSV files by clicking on the link in the admin view. Google just gives you a 4xx page.
  2. On stage, which has this deployed, follow (roughly) the logic in https://github.com/open-craft/accounting/blob/master/accounting/transferwise/tasks.py#L33 to create the bulk payment, create the CSV version, and upload to Google Drive (you can skip the email bit of course).
  3. Go into the billing console and see the CSV link for the new bulk payment you created. Click it and notice that it works without 4xx errors.

Merge request reports