-
Notifications
You must be signed in to change notification settings - Fork 0
Search function #2
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
Comments
Probably a good idea to start with this and eventually create a general search system allowing for multiple filters such as searching by wads and such. To give an idea, the current system fetches a list of SQIDs that can be decoded into an IP and port:
Then a list of servers is requested based on how many servers are visible in the virtual list that are unfetched:
Finally the servers are patched with their new data and the list can show properly. You also got another step when you open the server details but this is unrelated. I want to fetch as little data as possible in order to keep loading time at a minimum. What I could do is send a search criteria to the server, have the server filter the servers in memory, and then return just the SQID from these servers. Then the frontend can update the virtual list, which in turn triggers and asks for the list of servers which is now updated to show only the servers that meet the criteria. I believe the fetching of servers works fine even if part of these servers are already fetched. The main thing I need to implement is a way to indicate what servers should actually shown and which ones are fetched/unfetched. Currently I just have a single list of fetched/unfetched servers and I use this for both the fetching of new servers and showing the servers. I can't use this because there would be no way to properly indicate what servers to show without throwing away the whole list. |
With 42eb7f9 stage 1 of the search feature has now been properly implemented. |
Is your feature request related to a problem? Please describe.
Describe the solution you'd like

UI side
Technical side
Describe alternatives you've considered
The text was updated successfully, but these errors were encountered: