search

banco-alimentar / alimentestaideia.pt

Site doações alimente esta ideia
http://alimentestaideia.pt
Other
10 stars 0 forks source link

48 SqlException in last 24h, DonorAddresses cannot be NULL #711

Closed tiagonmas closed 1 year ago

tiagonmas commented 1 year ago

Describe the bug "An error occurred while saving the entity changes. See the inner exception for details. Cannot insert the value NULL into column 'Address1', table 'alimentestaideia.core.dbo.DonorAddresses'; column does not allow nulls. INSERT fails. "

BancoAlimentar.AlimentaEstaIdeia.Repository.PaymentNotificationRepository.AddEmailNotification

As seen in AppInsights

guerrerotook commented 1 year ago

I already push some code to try to mitigate this.

tiagonmas commented 1 year ago

when /where did you push the code to ? I checked master and dev and last updates where from 2 days ago before this happened ?

guerrerotook commented 1 year ago

My last try didn't work. I just this commit to try to solve the issue.

https://github.com/banco-alimentar/alimentestaideia.pt/commit/2c67faa1f17ecc0d1524771bd0ca7d453dbcff9d

I don't know why still complaints that DonorAddress is null when I'm not inserting anything new in the database. Anyway this is an small change.

tiagonmas commented 1 year ago

Trying to push this into preprod, but pipeline is failing on Publish

guerrerotook commented 1 year ago

Changed the way we are getting those entities when creating the notification. Now instead of getting these values from the database I force to get a WebUser and BasePayment that already exits. Hope this helps solve this issue that still showing in Applicaiton Insights.

tiagonmas commented 1 year ago

I dont see any more issues in the last 24h. I think we can close this one. Closing ...