Giveth-dapp: Milestone "Paid" doesnt connect to mainnet payment... only rinkeby

[GriffGreen]

https://beta.giveth.io/campaigns/5d110631994eac53adc8f21e/milestones/5ff3daeb2d93392e76e6b738

This milestone says Paid... BUT IT LIES!!! ;-) It was never paid... There probably could be a few issues out of this one... its a big fail of the system

see the https://bridge.beta.giveth.io/

[aminlatifi]

It has paid by this tx
https://etherscan.io/tx/0xa58a3f60cfb8b717482a01bc2a50ace9a2a237c8b1237bd3b9442f62aaae93a1

[aminlatifi]

I manually patched the db and could create this

@GriffGreen @bowensanders Is it enough?

[aminlatifi]

We should rewrite bridge monitor, it's design doesn't save events and not allow make an event be replayed!

[aminlatifi]

@GriffGreen I guess this issue is over... we have fixed the issue by patching bridge monitor manually. We should rewrite most of it to make it more reliable

[GriffGreen]

This should say "Payment Requested" when sent to the Bridge

This should only say paid for capped milestones if the payment has been sent on mainnet

[mohammadranjbarz]

@aminlatifi
Could you check and tell me if the exported csv is ok or need some changes:
common stack campaign donations.csv.zip

https://docs.google.com/spreadsheets/d/1PYTEn7AiN_rg3Q3_BpqWDBH-FoWVBGVCefSGcgnAUOU/edit?usp=sharing

[aminlatifi]

@aminlatifi
Could you check and tell me if the exported csv is ok or need some changes:
common stack campaign donations.csv.zip

https://docs.google.com/spreadsheets/d/1PYTEn7AiN_rg3Q3_BpqWDBH-FoWVBGVCefSGcgnAUOU/edit?usp=sharing

Thanks @mohammadranjbarz, just fix time format and and remove the bridge status column. Just having time and transaction means it has been paid and status column is not necessary

[aminlatifi]

New entry paymentTransactionHash is added to bridge monitor api output, it's added just for payments after 2021 Jan 30
https://feathers.bridge.beta.giveth.io/payments?event.returnValues.reference=0xaf2b14cd98187352bc1dbf8e44c758ee552d0936dacb9d1aad28d197fe5e0566

[aminlatifi]

New entry paymentTransactionHash is added to bridge monitor api output, it's added just for payments after 2021 Jan 30
https://feathers.bridge.beta.giveth.io/payments?event.returnValues.reference=0xaf2b14cd98187352bc1dbf8e44c758ee552d0936dacb9d1aad28d197fe5e0566

Older transactions have it now too

[aminlatifi]

The backend part is implemented