View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006896 | Taler | obsolete component | public | 2021-05-29 21:32 | 2021-08-24 14:22 |
| Reporter | Christian Grothoff | Assigned To | MS | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | high | Severity | tweak | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | i7 | OS | Debian GNU/Linux | OS Version | squeeze |
| Product Version | git (master) | ||||
| Target Version | 0.8 | Fixed in Version | 0.8 | ||
| Summary | 0006896: 'wire transfer' dialog ugly and broken | ||||
| Description | The new dialog for 'manual' wire transfers between accounts is broken. I can try to type in a payto URI like payto://x-taler-bank/bank.test.taler.net/exchange?message=foo&amount=TESTKUDOS:1.0 but it does not work, I just get a (badly rendered) error message. Also, in the past we had a much nicer dialog that auto-completed account names, and didn't expect me to re-type the bank base URL inside of a payto URL or all of the other syntactic gunk of a payto://-URI. It's OK to _also_ support payto://-URIs, but we should not ONLY support those. And, they must then actually work. I'm fine with this functionality having moved onto a separate page, but it should at least (1) still be easy to do such a transfer, and (2) not require payto:// (it can be an option). | ||||
| Tags | No tags attached. | ||||
| The "payto" address you wrote has the money receiver specified as a name, whereas the current version requires a number. I tried it right now on bank.test.taler.net by replacing "exchange" with a account _number_, and it worked. That said, is the error message you mention that one rectangular balloon saying "please match the requested format"? Or it is something else? | |
| Yes, that's the error message. It should be placed more nicely. Also, users have no good way to learn their account numbers, and in principle payto://-URIs can and should use the names of the accounts, so we should support using the names. And again, it would be best to have entering a payto://-URI as one option, but to generally have the simpler "target amount + account (name) + subject" entry back as well. | |
| Up to this commit: 9d87bacdb483e2ac74da1de7b6afc2833c78f5f4, the bank allows the specification of payment receiver via their usernames. | |
| The bug is mostly addressed, except that it doesn't autocomplete on account names. That's intentional, for the following reasons: 1. Why telling users which other users exist in the system? 2. Requires non trivial logic for a little gain. 3. Placing a element carrying a huge list of usernames next to the wire transfer form is eye-unfriendly. If OK with the three points above, this bug can be closed. | |
| Sounds good. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-05-29 21:32 | Christian Grothoff | New Issue | |
| 2021-05-29 21:32 | Christian Grothoff | Status | new => assigned |
| 2021-05-29 21:32 | Christian Grothoff | Assigned To | => MS |
| 2021-05-29 22:09 | MS | Note Added: 0017935 | |
| 2021-05-29 22:35 | Christian Grothoff | Note Added: 0017936 | |
| 2021-06-02 08:55 | MS | Note Added: 0017940 | |
| 2021-06-03 08:14 | MS | Note Added: 0017941 | |
| 2021-06-03 08:25 | MS | Status | assigned => feedback |
| 2021-06-03 09:11 | Christian Grothoff | Status | feedback => informing |
| 2021-06-03 09:11 | Christian Grothoff | Resolution | open => fixed |
| 2021-06-03 09:11 | Christian Grothoff | Fixed in Version | => 0.9 |
| 2021-06-03 09:11 | Christian Grothoff | Note Added: 0017942 | |
| 2021-07-30 11:57 | Christian Grothoff | Fixed in Version | 0.9 => 0.8.1 |
| 2021-07-30 11:59 | Christian Grothoff | Target Version | 0.9 => 0.8.1 |
| 2021-07-30 12:02 | Christian Grothoff | Fixed in Version | 0.8.1 => 0.8 |
| 2021-07-30 12:02 | Christian Grothoff | Target Version | 0.8.1 => 0.8 |
| 2021-08-24 14:22 | Christian Grothoff | Status | informing => closed |
| 2022-08-23 18:26 | Christian Grothoff | Category | bank (demonstrator) => py bank (demonstrator, obsolete) |
| 2023-12-03 00:23 | Christian Grothoff | Category | py bank (demonstrator, obsolete) => obsolete componet |
| 2023-12-11 19:08 | Florian Dold | Category | obsolete componet => obsolete component |