querybuilder icon indicating copy to clipboard operation
querybuilder copied to clipboard

Allow returning columns in Insert / Update clauses

Open dgadelha opened this issue 4 years ago • 16 comments

Implements an argument in Insert/Update-clauses related functions to allow returning columns directly through SQL

Compatibility:

  • [x] PostgreSQL
  • [x] MSSQL

A generic function was created to allow RDBMS-specific compilers (in this case, PostgreSQL and MSSQL) to handle special cases like this.

Fixes #8

dgadelha avatar May 10 '20 08:05 dgadelha

@dgadelha Would it be possible to extend this to also cover delete queries as requested in #133

@ahmad-moussawi Would be fantastic to get this merged as I currently have to code up and run select queries after each of the insert/updates.

b-twis avatar Jun 08 '20 08:06 b-twis

@ahmad-moussawi I guess the only blocker here would be to address the merge conflicts?

b-twis avatar Jun 12 '20 11:06 b-twis

@b-twis once the package maintainer shows up ensuring this PR will get some attention, I can do that

It's been a month since it was opened and no review. I can do what you asked but I need to make sure I won't waste my time

dgadelha avatar Jun 12 '20 22:06 dgadelha

Is there any update to this particular request? This would be an amazing addition to this package.

BrianPainter88 avatar Aug 18 '20 18:08 BrianPainter88

@ahmad-moussawi Would it be possible for you to find time to review this change?

b-twis avatar Sep 26 '20 07:09 b-twis

Bumpy bump 🛺.

graknol avatar Nov 17 '20 08:11 graknol

What needs to be done to move this PR forward?

ianclegg avatar Feb 10 '21 10:02 ianclegg

@ianclegg the PR have conflicts with the master branch

ahmad-moussawi avatar Feb 11 '21 15:02 ahmad-moussawi

Bump

aamcatamney avatar May 01 '21 22:05 aamcatamney

When will the PR be merged?

zh6335901 avatar May 19 '21 03:05 zh6335901

Bump

d-chirkov avatar May 19 '21 14:05 d-chirkov

Bump

aamcatamney avatar Aug 04 '21 22:08 aamcatamney

bump

iulian0512 avatar Mar 22 '22 15:03 iulian0512

Are there any plans to finish these changes and merge into the master?

deg-mallard avatar Oct 18 '23 13:10 deg-mallard

This looks great, wonder if this will ever get merged?

JakeAlmer avatar Oct 30 '23 15:10 JakeAlmer

Another upvote for merging this PR. This feature is crucial.

EricNgo1972 avatar Oct 31 '23 16:10 EricNgo1972