Giter Site home page Giter Site logo

Comments (3)

Reckless-Satoshi avatar Reckless-Satoshi commented on June 20, 2024 2

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!

from robosats.

Renato1478 avatar Renato1478 commented on June 20, 2024 2

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.

from robosats.

Reckless-Satoshi avatar Reckless-Satoshi commented on June 20, 2024 1

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.

from robosats.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.