-
Notifications
You must be signed in to change notification settings - Fork 820
Added copy as insert and copy as update #658
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
59b43e2 to
fedcc6a
Compare
|
@sosedoff updated the commit with requested changes. Also make error when table is not selected. Could not find another way to figure out the table name. |
|
Ping @sosedoff ? |
|
This looks good so far. One thing that may need to be addressed -- context menu does not work correctly when in query editor. Steps to reproduce:
Id suggest we disable the context menu options on sql query results page. |
|
Hello, |
|
Closing for the time being. Im happy to revisit if OP is interested in continuing the work. |
|
Yes im interested @sosedoff |
|
New PR here #730 |
|
ping @sosedoff ? |
Fixes #649