Skip to content
This repository was archived by the owner on Apr 17, 2020. It is now read-only.

Conversation

joelmukuthu
Copy link
Collaborator

@joelmukuthu joelmukuthu commented Sep 18, 2018

Closes knorm/knorm#55
Closes knorm/knorm#54

Requires PostgreSQL 9.5: jsonb_set

joelmukuthu added a commit to knorm/knorm that referenced this pull request Sep 21, 2018
Changes necessary to support JSON-patching in @knorm/postgres.
Ref: knorm/postgres#12
joelmukuthu added a commit to knorm/knorm that referenced this pull request Sep 21, 2018
Changes necessary to support JSON-patching in @knorm/postgres.
Ref: knorm/postgres#12
joelmukuthu added a commit to knorm/knorm that referenced this pull request Sep 21, 2018
Add:

- Query.prototype.getRowValue
- Query.prototype.getCastFields

Necessary to support JSON-patching in @knorm/postgres.

Ref: knorm/postgres#12
@joelmukuthu joelmukuthu force-pushed the feat/json-patching branch 2 times, most recently from 846c32c to d78e338 Compare September 22, 2018 00:03
@coveralls
Copy link

coveralls commented Sep 22, 2018

Pull Request Test Coverage Report for Build 110

  • 23 of 24 (95.83%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.6%) to 99.06%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/KnormPostgres.js 23 24 95.83%
Totals Coverage Status
Change from base Build 100: -0.6%
Covered Lines: 206
Relevant Lines: 207

💛 - Coveralls

Introduces a `patch` query option that accepts either `true`,
a string field-name, or an array of field-names. The field-names
define what fields in the update data to patch, or all if `true`
is passed.

Note that this is only supported for `json` and `jsonb` fields.
@joelmukuthu joelmukuthu changed the title wip: feat: support json-patching feat: support json-patching for updates Sep 23, 2018
@joelmukuthu joelmukuthu merged commit 728c122 into master Sep 23, 2018
@joelmukuthu joelmukuthu deleted the feat/json-patching branch September 23, 2018 21:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants