blockmason / lndr

Lending on the Blockchain
https://lndr.io/
9 stars 6 forks source link

Added migration to modify paypal_requests table and updated create_tables.sql #193

Closed willbach closed 6 years ago

willbach commented 6 years ago
  1. Problem: The Primary Key for paypal_requests should not have been friend it should have been (friend, requestor) to allow for multiple requests for a single friend, but not from the same requestor. Jira: https://blockmason.atlassian.net/secure/RapidBoard.jspa?rapidView=9&projectKey=ENG&modal=detail&selectedIssue=ENG-147
  2. Solution: Run a DB migration to update the paypal_requests table. This can be run on the current production DB without issue.
  3. No concerns
  4. The purpose of this PR is to record the DB change, the real change to the server will come from running the new migration, so there are no blockers to merge.
canterberry commented 6 years ago

Migration performed successfully. Merging.