-
Notifications
You must be signed in to change notification settings - Fork 77
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
Change offers TTL policy #2625
Comments
Changed title to make it easier for devs to quickly get what its about... |
21 is a beautiful round number indeed @suddenwhipvapor :) I'm happy to have a look into this, to round up your idea:
please confirm this is correct? thanks! |
yes, that's correct! |
The easiest way is to reduce the TTL to something like 1 day and do a refresh when the maker is online. So if maker was not online in that time the offer is gone, otherwise it gets refreshed before expired. In Bisq1 that's every 5 minutes but here it would be enough to use TTL/2 or so. |
okay, that's likely the most effective approach with code efficiency in mind. I would think that for those users with several open offers, having to reactivate each one after every restart will become quite annoying though. |
Awesome guys, I'm happy to add this to my pipeline this month 💪 |
In 2.1.1 there will be a "Show my offers only" checkbox. |
Keeping all offers indistinctly online for a total of 15 days even when the maker is offline could become bad UX, where possible takers have to deal with one-shot users who make an offer and then go away to never return, and serious 24/7 makers have to recreate their own offers every 15 days no matter what.
I suggest that there is an offline-grace-period for makers who cannot continuously be online, let's say 24hrs before which your offer will not be deleted, so the offerbook is pruned of dead offers where maker cannot check in daily, and the time-to-live for all other offers, where the maker was online at least daily, is increased, to at least 21 days if we want to use a nice round number.
The text was updated successfully, but these errors were encountered: