opensearch-project / opensearch-project/sql-odbc
[FEATURE] Add Windows Dialog Support for Serverless
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 10
- Forks
- 25
- PR merge metrics
- No merged PRs in 30d
Description
Is your feature request related to a problem?
Support for OpenSearch Serverless was added in #70. This did not including updating the Windows Dialog GUI to support the new ServerlessOverride configuration option. While this is not strictly necessary, it would improve the user experience.
What solution would you like?
I have no experience in this domain, but it seems like we need to:
- Update
resource.hto add anIDC_SERVERLESS_OVERRIDEconstant - Update
opensearch_odbc.rcto add an "Serverless Override" button to the Windows dialog. - Update
dconn_FDriverConnectProcindrvconn.cto correctly set focus for serverless override. - Update
dlg_wingui.c:SetDlgStuffandGetDlgStuffto populate dialog from theConnInfoconfiguration struct and populate theConnInfostruct from the dialog, respectively.
What alternatives have you considered?
None
Do you have any additional context?
None
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with resource.h and opensearch_odbc.rc to see how existing Windows dialog controls are declared and laid out. Trace dconn_FDriverConnectProc and the SetDlgStuff/GetDlgStuff functions in dlg_wingui.c, then wire the ServerlessOverride value through the dialog and ConnInfo structure. Done means the Windows dialog exposes and preserves the Serverless Override setting with correct focus behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- desktop
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100