# WU added or Bisq Fork? **URL:** \ **Category:** Development\ **Created:** [October 20, 2017, 1:33pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194 "2017-10-20T13:33:23Z")\ **Posts on this page:** 20\ **Page:** 3
**Author:** ![diegotco](https://bisq.community/user_avatar/bisq.community/diegotco/32/1106_2.png) [@diegotco](https://bisq.community/u/diegotco)\ **Post date:** [October 30, 2017, 1:49pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/41 "2017-10-30T13:49:36Z")
> [@ManfredKarrer](#): > > We can use the P2P network to send data and if the peer is offline to store it in the P2P network as “mailbox message”, but that has DDoS risks if we allow larger files Can we add some descentralized service to store data where the buyer have to allow check the receipt, maybe sia or storj? ---
**Author:** ![Nolaan](https://bisq.community/user_avatar/bisq.community/nolaan/32/1100_2.png) [@Nolaan](https://bisq.community/u/Nolaan)\ **Post date:** [October 30, 2017, 1:50pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/42 "2017-10-30T13:50:36Z")
> [@diegotco](#): > > maybe sia or storj Are they working properly, last time I checked it was missing the basic features… ---
**Author:** ![diegotco](https://bisq.community/user_avatar/bisq.community/diegotco/32/1106_2.png) [@diegotco](https://bisq.community/u/diegotco)\ **Post date:** [October 30, 2017, 1:57pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/43 "2017-10-30T13:57:57Z")
> [@ManfredKarrer](#): > > In the screenshot you have only full name. Is that enough for WU? Yes, it is 🙂 Someone can made mistakes on the name/lastname of the seller. He/her need to comeback to WU and fix that. That is why the seller need to see the receipt photo. It is necessary to know the country sender and the amounto of money too 🙂 ---
**Author:** ![diegotco](https://bisq.community/user_avatar/bisq.community/diegotco/32/1106_2.png) [@diegotco](https://bisq.community/u/diegotco)\ **Post date:** [October 30, 2017, 2:09pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/44 "2017-10-30T14:09:47Z")
> [@Nolaan](#): > > I’m not sure if it’s required but sometimes cashiers do ask the amount sent, and this could be a proof for this amount, especially with larger sums + fees you could be easily missing 10%. @diegotco to confirm. Yes, they allwas ask about the amount of money to collect. But seller receive all the money. The buyer has to spend Amount of the buy + Fees. And is not more than 10% here in Latinamerica. Even there is a service here where you can send money cheaper but the other person has to wait for 24 hours to collect money Even with 10% of fees, WU is cheaper service for everyone here and very easy 🙂 Because you do not need banks accounts, you don’t need to have a credit card and of course you don’t need to pay 40% Localbitcoins overprice hehe ---
**Author:** ![diegotco](https://bisq.community/user_avatar/bisq.community/diegotco/32/1106_2.png) [@diegotco](https://bisq.community/u/diegotco)\ **Post date:** [October 30, 2017, 2:14pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/45 "2017-10-30T14:14:46Z")
Yes, even [K.im](http://K.im) (Kim dot com services is going to run on them) ---
**Author:** ![Nolaan](https://bisq.community/user_avatar/bisq.community/nolaan/32/1100_2.png) [@Nolaan](https://bisq.community/u/Nolaan)\ **Post date:** [October 30, 2017, 2:23pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/46 "2017-10-30T14:23:38Z")
Okay I was probably skeptical because of madesafecoin; endless whitepaper and no real progress. ---
**Author:** ![ManfredKarrer](https://bisq.community/user_avatar/bisq.community/manfredkarrer/32/26_2.png) [@ManfredKarrer](https://bisq.community/u/ManfredKarrer)\ **Post date:** [October 30, 2017, 4:14pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/47 "2017-10-30T16:14:43Z")
> [@Nolaan](#): > > There room for automation here. It would be good if in the long term we could have unit tests + functional tests for the whole program. Yes would be great but its quite an effort and challenge. For testing a whole process like a trade it requires mocking the P2P network, the BTC part (thats the hardest) and the UI interactions. Once we have more resources it would be definitely cool to get more automated tests for those areas but so far it would eat up too much resources we need for other stuff. ---
**Author:** ![ManfredKarrer](https://bisq.community/user_avatar/bisq.community/manfredkarrer/32/26_2.png) [@ManfredKarrer](https://bisq.community/u/ManfredKarrer)\ **Post date:** [October 30, 2017, 4:17pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/48 "2017-10-30T16:17:26Z")
@Nolaan I just saw you forked from master. I would recommend to use the Development branch as there are a lot of changes and merging will be easier. You need to recompile to get the correct Protobuffer generated classes when changing to dev branch. There will also very soon (today) be merged a PR where we update to another Tor library which will remove the 4 Tor modules. Maybe those have broken your UML? ---
**Author:** ![ManfredKarrer](https://bisq.community/user_avatar/bisq.community/manfredkarrer/32/26_2.png) [@ManfredKarrer](https://bisq.community/u/ManfredKarrer)\ **Post date:** [October 30, 2017, 4:24pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/49 "2017-10-30T16:24:02Z")
I am also skeptical if they are “production ready”. Last time I tried it out it was super slow. It would also cause a lot of engineering effort to integrate that for such a minor feature. If the photo upload is highly demanded we can still do it in our P2P network, I just try to be conservative to only add the absolute needed to not overload it and not expose it to too many attack surfaces. Another simple approach is to use any http photo upload service and just send the link. The service should support Tor (no cloudflare), then we could even integrate it to show the image in the app. Of course carries privacy issues if the image contains personal data. Encrypting it and decrypting the uploaded file before display might be another approach… ---
**Author:** ![Nolaan](https://bisq.community/user_avatar/bisq.community/nolaan/32/1100_2.png) [@Nolaan](https://bisq.community/u/Nolaan)\ **Post date:** [October 30, 2017, 6:16pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/50 "2017-10-30T18:16:38Z")
> [@ManfredKarrer](#): > > Yes would be great but its quite an effort and challenge Slowly but surely 😉 > [@ManfredKarrer](#): > > Maybe those have broken your UML? No ideas, seems to me like a recursion in network/p2p/network. I couldn’t investigate more. I’ll try again after the PR merge. This is what I used to understand how an offer and a payment account relate to each other. I didn’t go to the details because of the bug. I also have a sequence diagram somewhere. ![payment_and_offer_class_diagram](https://bisq.community/uploads/default/original/2X/3/32c431ff6a3c0a66d3ef1df23117029f3f7175fa.png) ---
**Author:** ![ManfredKarrer](https://bisq.community/user_avatar/bisq.community/manfredkarrer/32/26_2.png) [@ManfredKarrer](https://bisq.community/u/ManfredKarrer)\ **Post date:** [October 30, 2017, 8:44pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/51 "2017-10-30T20:44:12Z")
Yes that’s a good start: easiest is if you search for occurrence of other accounts (e.g. sepa) and see where and what to change/add. There are some view classes, domain, protobuffer and translation strings. New classes to create are: - WesternUnionAccountPayload - WesternUnionAccount - WesternUnionForm - WesternUnionValidator The classes you need to touch are: FiatAccountsView BuyerStep2View PaymentAccountFactory CoreProtoResolver Protobuffer: - pb.proto translations files (only english file): - search for CASH\_DEPOSIT=Cash Deposit ---
**Author:** ![Nolaan](https://bisq.community/user_avatar/bisq.community/nolaan/32/1100_2.png) [@Nolaan](https://bisq.community/u/Nolaan)\ **Post date:** [October 31, 2017, 5:08am UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/52 "2017-10-31T05:08:25Z")
Cool looks like what I did, so I will just change the logic in the code behind and I’m ready to submit the PR. ---
**Author:** ![ManfredKarrer](https://bisq.community/user_avatar/bisq.community/manfredkarrer/32/26_2.png) [@ManfredKarrer](https://bisq.community/u/ManfredKarrer)\ **Post date:** [October 31, 2017, 2:21pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/53 "2017-10-31T14:21:09Z")
Cool! Please cross check with @diegotco if the provided fields are sufficient and if the way how the receipt and/or tracking number is exchanged is correct. I will not have time to investigate WU process and I am not familiar with it. ---
**Author:** ![marc](https://bisq.community/letter_avatar/marc/32/6_22734026db12803ebb3e059622ac3534.png) [@marc](https://bisq.community/u/marc)\ **Post date:** [November 1, 2017, 5:03pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/54 "2017-11-01T17:03:00Z")
> [@ManfredKarrer](#): > > With photos we could handle it like with cash deposit to delegate it to users to email receipt. I’d go with that suggestion if it’s needed at all, as it seems to have the least friction and low risk and would provide a safety net in case someone makes a mistake sending the money as @diegotco mentioned. ---
**Author:** ![diegotco](https://bisq.community/user_avatar/bisq.community/diegotco/32/1106_2.png) [@diegotco](https://bisq.community/u/diegotco)\ **Post date:** [November 7, 2017, 12:48pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/55 "2017-11-07T12:48:53Z")
Also, photo is important to validate payment because buyer may show a whatever MTCN number that already has been collected but it doesn’t means that the sender/receiver names, amount and country are ok. That’s why we need to use only this WU URL: [https://www.westernunion.com/es/en/track-transfer.html](https://www.westernunion.com/es/en/track-transfer.html) This website ask the amount in the receiver’s country currency 🙂 ---
**Author:** ![ManfredKarrer](https://bisq.community/user_avatar/bisq.community/manfredkarrer/32/26_2.png) [@ManfredKarrer](https://bisq.community/u/ManfredKarrer)\ **Post date:** [November 7, 2017, 9:24pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/56 "2017-11-07T21:24:32Z")
@Nolaan I am close to the release code freeze. If you can deliver before 10. November and there are no issues with the PR it likely can go into the release, otherwise into the next release. ---
**Author:** ![Nolaan](https://bisq.community/user_avatar/bisq.community/nolaan/32/1100_2.png) [@Nolaan](https://bisq.community/u/Nolaan)\ **Post date:** [November 8, 2017, 1:41pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/57 "2017-11-08T13:41:04Z")
I’m hurrying up, didn’t have as much time I wanted. Anyway I’ve already rebased onto the Development branch and started the tests on regtest network. If I’m good enough I PR tonight. ---
**Author:** ![diegotco](https://bisq.community/user_avatar/bisq.community/diegotco/32/1106_2.png) [@diegotco](https://bisq.community/u/diegotco)\ **Post date:** [November 8, 2017, 2:43pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/58 "2017-11-08T14:43:17Z")
Thank you @Nolaan all the community are in debt with you. May I have you btc address? I want you be rewarded by Latin American community ---
**Author:** ![ManfredKarrer](https://bisq.community/user_avatar/bisq.community/manfredkarrer/32/26_2.png) [@ManfredKarrer](https://bisq.community/u/ManfredKarrer)\ **Post date:** [November 9, 2017, 12:32am UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/59 "2017-11-09T00:32:15Z")
Due the B2X canceling we are a bit more relaxed with release. I can help if there are small issues open and with testing. You can make a PR as soon its more or less ok and i will check. ---
**Author:** ![ManfredKarrer](https://bisq.community/user_avatar/bisq.community/manfredkarrer/32/26_2.png) [@ManfredKarrer](https://bisq.community/u/ManfredKarrer)\ **Post date:** [November 11, 2017, 8:05pm UTC](https://bisq.community/t/wu-added-or-bisq-fork/3194/61 "2017-11-11T20:05:51Z")
@diegotco @Nolaan: I merged the PR from @Nolaan and added the missing parts. Here are screenshots from the relevant screens. Please check and let me know if those data and the trade process is correct with WU. There is a restriction that you can only send money with the same currency. To support multi currencies would require more effort. Account setup: ![23](https://bisq.community/uploads/default/original/2X/6/6e7ae823e33b36fe7e8ac75b8765e80e3be1df5a.png) Save account info: ![30](https://bisq.community/uploads/default/original/2X/7/742cc0e7483a635b42cdbc485998a54acd350aaa.jpg) Account view: ![41](https://bisq.community/uploads/default/original/2X/f/f9d0fffc6d97495b67a73d27bd7a742a90d3dd8e.png) Send funds info 1: ![39](https://bisq.community/uploads/default/original/2X/2/244a7858dd1fd0dd8f4bf396d9363901e36a0bca.jpg) Send funds info 2: ![45](https://bisq.community/uploads/default/original/2X/d/d8a2c658fcc4abfc656425e65db37c68289134b8.png) Received funds popup: ![09](https://bisq.community/uploads/default/original/2X/b/bd2a0cf20d85338d1efafc0b8a9cdefcb5b6a529.jpg) Received funds data for pick up: ![49](https://bisq.community/uploads/default/original/2X/6/6407c07558a019a1bdd208bc42e832768e48bd9a.png) [Previous page](https://bisq.community/t/wu-added-or-bisq-fork/3194.md?page=2) [Next page](https://bisq.community/t/wu-added-or-bisq-fork/3194.md?page=4)