search

blockmason / lndr

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

Allow a single user to have multiple channel ids 345325d #200

Closed willbach closed 6 years ago

willbach commented 6 years ago
  1. Jira: https://blockmason.atlassian.net/browse/LNDR-8
  2. Problem: The Lndr DB has a unique constraint on channel_id in the push_data table that throws a 500 error when a user tries to register a new channel_id
  3. Solution: Change the primary key of the push_data table and modify the upsert query to allow a user to have multiple channel_ids.
  4. No concerns
  5. Before merging, the new migration file should be run: lndr-backend/db/migrations/2018-10-08T10.00.00Z_change_push_data_pk.sql
canterberry commented 6 years ago

Looks good, apart from the migration. Get that fixed up and we're good to migrate then merge.

canterberry commented 6 years ago

:+1: LGTM. Going to run this migration then merge.