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

feature: password protected trades/offers #714

Open
LeoSpyke opened this issue Jul 12, 2023 · 3 comments
Open

feature: password protected trades/offers #714

LeoSpyke opened this issue Jul 12, 2023 · 3 comments
Assignees
Labels
enhancement 🆙 New feature or request

Comments

@LeoSpyke
Copy link
Contributor

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.

@Reckless-Satoshi Reckless-Satoshi added the enhancement 🆙 New feature or request label Jul 13, 2023
@Reckless-Satoshi
Copy link
Collaborator

Reckless-Satoshi commented Jul 13, 2023

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!

@Renato1478
Copy link
Contributor

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.

@Reckless-Satoshi
Copy link
Collaborator

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 🆙 New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants