search

Goodler / code

Goodler
Apache License 2.0
0 stars 2 forks source link

Error message when deleting a charity's address. #584

Open galinafed opened 6 years ago

galinafed commented 6 years ago

screen shot 2017-12-21 at 10 00 12 am

screen shot 2017-12-21 at 9 59 43 am

screen shot 2017-12-21 at 9 59 25 am I updated an address for Lava Mae and wanted to delete an old address, it was showing error. I tried to click on Save and Save and Exit. Both actions produced an error message. screen shot 2017-12-21 at 9 59 11 am

DmitrysIF commented 6 years ago

Ок, гляну.

DmitrysIF commented 6 years ago

Удалить адрес нельзя, потому, что он уже используется в других вишлистах. Возникает конфликт, когда пользователь хочет удалить адрес, а система не даёт, так как это нарушит целостность базы данных. Мы внедряли защиту от этой ошибки, но она, как видно, не работает как положено.

Нужно вместо физического удаления делать логическое удаление адреса.

galinafed commented 6 years ago

Got it. Have to think about it then, company moved and that address is no longer a valid address. Not a big deal right now, but have to find a way to hide it.

DmitrysIF commented 6 years ago

@DeveloperRegistry Антон, что скажешь по этой проблеме?

DeveloperRegistry commented 6 years ago

You can implement a logical delete for the address table (consistent with other tables).

From: DmitrysIF notifications@github.com Sent: Thursday, December 21, 2017 11:03 AM To: Goodler/code Cc: Anton Spektorov; Mention Subject: Re: [Goodler/code] Error message when deleting a charity's address. (#584)

@DeveloperRegistryhttps://github.com/developerregistry Антон, что скажешь по этой проблеме?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/Goodler/code/issues/584#issuecomment-353431236, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADpTop_RmpzIYLsHmKXRP3DlchSRlyGlks5tCqtngaJpZM4RKKmJ.

DmitrysIF commented 6 years ago

Хорошо, я так и думал.