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

Move info from 'Peer Info' to 'Trade Information Screen' and 'Detail Data' #6036

Closed
pazza83 opened this issue Feb 10, 2022 · 2 comments · Fixed by #6053
Closed

Move info from 'Peer Info' to 'Trade Information Screen' and 'Detail Data' #6036

pazza83 opened this issue Feb 10, 2022 · 2 comments · Fixed by #6053

Comments

@pazza83
Copy link

pazza83 commented Feb 10, 2022

Background

Following a recent Bisq support call I was investigating why a user might have been able to take a SEPA offer of over 0.01 BTC without having a signed account.

AFAIK signed status for trades in disputes is not information that can be seen in the mediator Bisq instance. This hampered my investigation :(

This lead me to write the issue

Description

Currently the following are viable on the peer info screen

  • Onion address
  • Number of completed trades
  • Payment account age
  • Account signed age
  • Set tag for that peer

I propose removing from this screen the following:

  • Number of completed trades
  • Payment account age
  • Account signed age

This info should instead be added to:

  • Trade Information
  • Detail Data

The main reason for this is I think it would be beneficial for signed status to be shown in the trade information screen and be visible to mediators and arbitrators should the trade enter dispute resolution (currently signed status is not visible to mediators or arbitrators).

Signed status being visible to mediators and arbitrators would help ensure things such as trades were not being take over the btc limits by un-signed accounts, and help with the identification of any nefarious activity.

Also with the recent change to clear payment account payload info from closed trades it would seem to make sense to also clear the above information from the peer info screen.

I would propose that double clicking on an avatar just displays:

  • Onion address
  • Set tag for that peer

I would not propose making any changes to the avatar. Could still show number of completed trades in small circle as current.

Version

1.8.2

Steps to reproduce

Double click a peer's avatar to see what information is displayed. Information displayed varies depending on national payment method (signing required or not required) or if it is an altcoin trade.

Expected behaviour

Double clicking on a peers avatar just displays:

  • Onion address
  • Set tag for that peer

From the Trade InformationandDetail Data` screens the following information can be ascertained*:

  • Number of completed trades
  • Payment account age
  • Account signed age

*My preference would be for the above to be a snapshot at the time the trade being completed as opposed to updating as time progresses, number of trades increase (as happens now).

@ghost
Copy link

ghost commented Feb 12, 2022

I am working on this.

@ripcurlx ripcurlx assigned ghost Feb 14, 2022
@ghost
Copy link

ghost commented Feb 16, 2022

After discussing with @pazza83, we agreed to include signed status when applicable in the Dispute Detail and Trade Detail screens, where currently only account age is shown.

We opted not to remove data from the Peer Info screen because it is used by traders for decision making.

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

Successfully merging a pull request may close this issue.

1 participant