Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Bank transfer 'Copy BSB number' bug #5200

Closed
Conza88 opened this issue Feb 17, 2021 · 9 comments · Fixed by #5231
Closed

Fix Bank transfer 'Copy BSB number' bug #5200

Conza88 opened this issue Feb 17, 2021 · 9 comments · Fixed by #5231

Comments

@Conza88
Copy link

Conza88 commented Feb 17, 2021

Steps to reproduce

  • Take offer
  • National Bank Transfer method
  • Go to pay trader, open trade ID details
  • Hit copy the :BSB code: number
  • *Instead it copies the BANK NAME, not the BSB code/number

Most certainly first world problems I know, but is a small fix hopefully.

May be present on OTHER bank payment methods as well.

image

@pazza83
Copy link

pazza83 commented Feb 17, 2021

Thanks for this I will take a look.

@BtcContributor
Copy link
Contributor

Not sure if I can reproduce that @Conza88, at least on 1.5.6.
If I click on the copy button I obtain the value of Bank name / BSB code together and it seems good to me.

Maybe it has already been fixed?

@Conza88
Copy link
Author

Conza88 commented Feb 23, 2021

Should it not just be the BSB number on its own?

You don't need the bank name when putting into transfer at all. The BSB and account number together look that up.

When I paste it currently into bsb box it pastes the stupid bank name instead and gets cut off

@BtcContributor
Copy link
Contributor

I think that the copy button means copy everything in the box and that's what is happening right now.
If just the second part of the box is needed instead, I can propose a fix for that.

@Conza88
Copy link
Author

Conza88 commented Feb 23, 2021 via email

BtcContributor added a commit to BtcContributor/bisq that referenced this issue Feb 24, 2021
@BtcContributor
Copy link
Contributor

Fixed @Conza88.
It is trivial to add other similar scenarios, if needed.

@Conza88
Copy link
Author

Conza88 commented Feb 24, 2021

Awesome. Yes, pretty simple.

Is there any reason any bank transfers need the other bank name?

@BtcContributor
Copy link
Contributor

Not sure about that.

@pazza83
Copy link

pazza83 commented Feb 28, 2021

Is there any reason any bank transfers need the other bank name?

@Conza88, good question.

I think it is standard practice when making payment to someone to know the bank you are transferring to. However really the info is not needed as you have the BIC / Sort Code / BSB Code etc.

That being said I do not think it really gives too much away as because you have the above you would be able to find their bank name anyway.

Let me know if you think it is / is not needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants