frectonz / sql-studio

SQL Database Explorer [SQLite, libSQL, PostgreSQL, MySQL/MariaDB, DuckDB]
https://sql-studio.frectonz.io/
MIT License
1.51k stars 26 forks source link

Feat/auto browser and bad query fallback #9

Closed Kinfe123 closed 1 week ago

Kinfe123 commented 1 week ago

This change include

frectonz commented 1 week ago

I like the automatically opening the URL thing, but i don't like the "No such resource returned ..." thing, i prefer the skeleton. If you remove the that i will merge this PR.

Kinfe123 commented 1 week ago

do you think skeleton would make sense for the users who aint know whether their query fails or not.. or we can stop the loading skeleon if the query fails to do so.

frectonz commented 1 week ago

Ok yeah, we should do this

Since i want people to directly type in their queries and have the data fetched automatically, i don't want them to be bombarded with a "No such resource error".

Kinfe123 commented 1 week ago

yeah we can handle that by debouncing but yeah it is better to return a nothing.

frectonz commented 1 week ago

Yeah but don't add debouncing. I want querying to feel extremely fast, deboucing will just add a delay. Since people are running the server on their own local machine, we don't care about how many times they hit the /query route. Debouncing would have made sense if we were calling an external API.

Kinfe123 commented 1 week ago

hmmm....makes sense , it is fixed by now