kndndrj / nvim-dbee

Interactive database client for neovim
GNU General Public License v3.0
617 stars 40 forks source link

Can the connection configuration file be added to be compatible with sqlls? #42

Open xYx-c opened 9 months ago

xYx-c commented 9 months ago

I want to use sqlls smart completion. But their configuration connection information is not compatible. I don't want to maintain two sets of roughly the same connection information. dbee must be in square brackets Expect to get the following connection information template

{
    "connections" : [{
      "id": "__master_connection_id_postgresqlpostgres__",
      "page_size": 20,
      "type": "postgres",
      "url": "postgres://postgres:pg_pass@localhost/pg_test?sslmode=disable",
      "name": "postgres-project",
      "adapter": "postgres",
      "host": "localhost",
      "port": 5432,
      "user": "postgres",
      "password": "pg_pass",
      "database": "pg_test",
      "projectPaths": ["/Users/joe-re/src/postgres_project"]
   }]
}

from translate.google

xYx-c commented 9 months ago

After updating neovim and dbee I got a new error

nvim:nvim_buf_is_valid exception: Wrong type for argument 1 when calling nvim_buf_is_valid, expecting Buffer

kndndrj commented 9 months ago

Hey, thanks for the suggestion. I think it makes sense to support sqlls in the future for sure. However, as of now I suspect quite a lot of people are using the existing file format, so I don't want to change it's format.

The way I see it, we can create a new separate source for sqlls. I'll add a label as an enhancement here, but I don't expect to be able to work on it any time soon.

If you wish you can look into lua/dbee/sources and try implementing the source yourself in your own configuration. I would also welcome a PR :)

xYx-c commented 9 months ago

@kndndrj nvim:nvim_buf_is_valid exception: Wrong type for argument 1 when calling nvim_buf_is_valid, expecting Buffer Is this error a problem with my configuration?