Skip to content

Commit

Permalink
fix(payments): fix missing metadata in postgres (#1309)
Browse files Browse the repository at this point in the history
  • Loading branch information
paul-nicolas committed Mar 6, 2024
1 parent b937044 commit 65c7c78
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 10 deletions.
Expand Up @@ -14,7 +14,6 @@ import (
type PaymentBatchElement struct {
Payment *models.Payment
Adjustment *models.PaymentAdjustment
Metadata []*models.PaymentMetadata
}

type PaymentBatch []PaymentBatchElement
Expand Down Expand Up @@ -42,11 +41,12 @@ func (i *DefaultIngester) IngestPayments(

for batchIdx := range batch {
payment := batch[batchIdx].Payment
metadata := batch[batchIdx].Metadata
adjustment := batch[batchIdx].Adjustment

if metadata != nil {
for _, data := range metadata {
if payment != nil {
allPayments = append(allPayments, payment)

for _, data := range payment.Metadata {
data.Changelog = append(data.Changelog,
models.MetadataChangelog{
CreatedAt: time.Now(),
Expand All @@ -57,10 +57,6 @@ func (i *DefaultIngester) IngestPayments(
}
}

if payment != nil {
allPayments = append(allPayments, payment)
}

if adjustment != nil && adjustment.Reference != "" {
allAdjustments = append(allAdjustments, adjustment)
}
Expand Down
Expand Up @@ -125,7 +125,7 @@ func (s *Storage) UpsertPaymentsMetadata(ctx context.Context, metadata []*models
Model(&metadata).
On("CONFLICT (payment_id, key) DO UPDATE").
Set("value = EXCLUDED.value").
Set("changelog = metadata.changelog || EXCLUDED.changelog").
Set("changelog = payment_metadata.changelog || EXCLUDED.changelog").
Exec(ctx)
if err != nil {
return e("failed to create metadata", err)
Expand Down
2 changes: 1 addition & 1 deletion components/payments/internal/models/payment.go
Expand Up @@ -109,8 +109,8 @@ type Payment struct {
DestinationAccountID *AccountID `bun:",type:character varying,nullzero"`

// Read only fields
Adjustments []*PaymentAdjustment `bun:"rel:has-many,join:id=payment_id"`
Metadata []*PaymentMetadata `bun:"rel:has-many,join:id=payment_id"`
Adjustments []*PaymentAdjustment `bun:"rel:has-many,join:id=payment_id"`
Connector *Connector `bun:"rel:has-one,join:connector_id=id"`
}

Expand Down

0 comments on commit 65c7c78

Please sign in to comment.