kndndrj / nvim-dbee

Interactive database client for neovim

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

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

xYx-c opened this issue · comments

commented

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

commented

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

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 :)

commented

@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?