robosats icon indicating copy to clipboard operation
robosats copied to clipboard

feature: password protected trades/offers

Open LeoSpyke opened this issue 2 years ago • 3 comments
trafficstars

Is your feature request related to a problem? Please describe. Sometimes it might happen that two actors want to perform a "private" trade, with pre-agreed conversion rates and currencies, but without giving up the functionality and "insurance" service offered by a third-party escrow.

Describe the solution you'd like The idea is to simply add a "Private trade" checkbox during the "Sell Order" creation which enables a password input field.

Describe alternatives you've considered None, I don't know if it is really feasible.

LeoSpyke avatar Jul 12 '23 13:07 LeoSpyke

Hey @LeoSpyke

This is indeed a feasible feature that should be easy to implement.

An straight forward way to go about it is to add 2 new fields to the Order model: is_private and password. If an order is_private == True, it should not be served via /api/book . Only a direct link should be able to access the order. Once a user opens the direct link, he should only be able to take it if he knows the password.

We could further simplify it if the /api/order/ endpoint accepts the password as an argument (e.g, /api/order/?order_id=9999&password=abcd) and if we can relay this argument directly from the frontend by using the route /order/9999/abcd. That way users do not need to share link and password, but only the link. And we do not need to create a new frontend component to enter the password (less work, gud!).

This feature will require both, new backend and frontend functionality. I am a bit at capacity lately, but happy to help if anyone wants to take the challenge!

Reckless-Satoshi avatar Jul 13 '23 08:07 Reckless-Satoshi

I think the buyer could have this option as well... I imagined like if you're in a multiplayer game looking for lobbies and there are "public" and "private" matches. The private ones have a password. Is that the idea @LeoSpyke ?

Idk if this is a lot for the escope of the application. I don't see any other DEX having this kind of feature. So maybe is worth collecting users opinions on this just like @Reckless-Satoshi did for the #1154 PR on Telegram.

Anyway, I'm willing to take this enhancement. Assign me if it's still desirable.

Renato1478 avatar Mar 15 '24 23:03 Renato1478

I think the buyer could have this option as well...

Yes, this should be a new field in the MakerForm under "Advanced Options" available for both, buy and sell orders.

Idk if this is a lot for the escope of the application. I don't see any other DEX having this kind of feature. So maybe is worth collecting users opinions on this just like @Reckless-Satoshi did for the #1154 PR on Telegram.

Mmm, I think it is potentially useful! It will be a niche feature though. Therefore, it is best if this option is kind of hidden and does not clutter the UI or makes it difficult to navigate for newbies.

Anyway, I'm willing to take this enhancement. Assign me if it's still desirable.

Awesome! We can turn it into a Sats rewarded task for 120K Satoshis. I am assigning you.

Reckless-Satoshi avatar Mar 18 '24 01:03 Reckless-Satoshi