-
Notifications
You must be signed in to change notification settings - Fork 63
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 request] Allow filter dates #37
Comments
That's a good idea. I would have to think about how I would integrate that into the UI. Moreover, I think this could actually help with the restriction on the amount of calls because (iirc) that information is returned before I have to make an API call to obtain the However, keep in mind I'll unfortunately be busy with other things until October, though. |
There could be a filter to every column, like if I only want forks that are no commits behind and at least 10 commits ahead. |
I think this would be the best approach. It would ultimately unclog the "Settings" dialog which itself could be used for some more complex related things (for example, I have this idea about offering different scanning strategies to users, and maybe having the default be an automatic swap between strategies based on the amount of forks of the queried repo). The right way to implement this would be to keep the entire result list in memory, and to only display a subset based on filtering preferences. The current implementation just throws away the data which doesn't meet the provided criterias, so it isn't flexible enough to support this kind of use-case. I'd be stoked if someone wanted to work on this. :) |
how about allow search only forks updated the last 1 day / 1 week / 1 month / 1 year / custom?
The text was updated successfully, but these errors were encountered: