Skip to content

Save knex instance to this.knex instead of this.db #43

@lorensr

Description

@lorensr

In order to:

  • avoid confusion between having multiple dbs: dataSources.db.* in resolvers and this.db.* in data source
  • more closely match knex docs (which say eg knex.select().from('users')), so we'd do

this.knex.select().from('users')
instead of
this.db.select().from('users')

https://github.com/cvburgess/SQLDataSource/blob/master/index.js#L17

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions