Skip to content

Binding parms UI - #578

Merged
SanjulaGanepola merged 9 commits into
mainfrom
BindParm
Sep 29, 2026
Merged

SanjulaGanepola merged 9 commits into
mainfrom
BindParm

Conversation

@buzzia2001

@buzzia2001 buzzia2001 commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Changes

This PR introduces the option to have a user interface for entering binding parameters.

How to test this PR

Open text editor and run: select * from qiws.qcustcdt where lstnam=:cognome or baldue=:baldue, and the run the query, you'll have a form where you can specify values:

image

Values are saved, so you don't have to refill the form again...
if you have ? and :var you'll have a form to fill like this one:

image

Checklist

  • have tested my change
  • have created one or more test cases
  • updated relevant documentation
  • Remove any/all console.logs I added
  • have added myself to the contributors' list in CONTRIBUTING.md

Closes #572

@buzzia2001 buzzia2001 self-assigned this Sep 24, 2026
@github-actions

github-actions Bot commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

👋 A new build is available for this PR based on 9b1d03b.

@buzzia2001 buzzia2001 added the enhancement New feature or request label Sep 24, 2026
@buzzia2001

Copy link
Copy Markdown
Member Author

@SanjulaGanepola to you

Comment thread src/views/results/binding.ts Outdated
Comment thread src/views/results/index.ts Outdated
Comment thread src/views/results/index.ts Outdated
Comment thread src/views/results/index.ts Outdated
Comment thread package.json Outdated
Comment thread src/views/results/binding.ts Outdated
Comment thread src/views/results/binding.ts Outdated
Comment thread package.json
Comment thread src/views/results/binding.ts
Comment thread src/language/sql/document.ts Outdated
@buzzia2001

Copy link
Copy Markdown
Member Author

@SanjulaGanepola all points have been addressed... let me know what do you think

@SanjulaGanepola SanjulaGanepola left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! It works great

@SanjulaGanepola
SanjulaGanepola merged commit 0fae420 into main Sep 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add prompting for Named Parameters for SQL Scripts

2 participants