blockchainprojects / bitshares-ui

Graphical User Interface for BitShares Blockchain
https://wallet.bitshares.org
MIT License
0 stars 1 forks source link

[14.5] #2244 Wallet Merchant #94

Open sschiessl-bcp opened 5 years ago

sschiessl-bcp commented 5 years ago

Original issue #2244

Task see https://github.com/bitshares/bitshares-ui/issues/2244#issuecomment-480789973

VerevkinAlexander commented 5 years ago

The issue is fixed, please see PR

sschiessl-bcp commented 5 years ago

Super!

Now I would like a new button there (upper right corner maybe) "Print receipt"

It creates a PDF (A4, like paper wallet), very simply, that looks something like this image (the total amount is obviosuly wrong atm).

Please allow for different assets to be invoiced (then we need a second row in the Receipt Total area), also the Transaction part can contain multiple transaction ids.

StarodubetsWork commented 5 years ago

@sschiessl-bcp can you help me? I stuck with a problem, I can `t get transaction id and receipt date. I only know how to get this info from the account history but do not know how to get it on transaction confirmation. Also after we received transaction confirmation app redirect us to another page and we have no possibility to click on the print button.
https://github.com/blockchainprojects/bitshares-ui/blob/1af9b15af21d360e1bf6059230adbcd6cbb3ac1a/app/components/Transfer/Invoice.jsx#L100

Please allow for different assets to be invoiced (then we need a second row in the Receipt Total area),

Not sure that understand you correctly. Would you mind to give me an example of the JSON file?

sschiessl-bcp commented 5 years ago

Comment the redirect out, it's not important atm.

Does the transfer that is being triggered contain a memo with the invoice number? If not, please change that the transfer contains it.

Then have the print receipt button unlock the wallet and then check the account history if there is a transfer with the specified invoice number in memo. If such a transfer is found (and the from and to accounts are also correct), assume this is the transfer for this invoice and show in pdf.

VerevkinAlexander commented 5 years ago

I`ve implemented it, please see my PR

sschiessl-bcp commented 5 years ago

For testing http://localhost:8080/invoice/Ch8t3vKjc9Ydc6qskfBoDZAen1fdUAtukiLyoeeG2veD1Dv6DzGDctrDoQkUB4dBMoQn8qoon3TgbdKHdJP9yFK3r443uTpEUYHPSbwUiWJRnwi1qRumDUidzwGySGGgpb3bonKJTBeYHYvDkraz7C72RXaPFXLujCDioJMrSzqJgvL3Rd1ajA8i1GQYRf9YQZT7qhYDLPy44zEZiRmumcgS5EKZbdbXHxrQwnVh62F18Zf5fCct9CZ25xwEEHJsasLHR7Q8aVkSqtoJTzmFqse3LTijugrj3tpH34h4r1k1hnaZQUmhk2yUjgfyimnETTUasbkNtNTh

sschiessl-bcp commented 5 years ago
VerevkinAlexander commented 5 years ago
  • the button is always grayed out for me

It should be enabled when the transaction is created because some info is missed until the transaction is not created. I can add a tooltip with explanation

VerevkinAlexander commented 5 years ago

image

VerevkinAlexander commented 5 years ago

image