-
Notifications
You must be signed in to change notification settings - Fork 492
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
feat(amazonq): support SQL conversions #5775
Conversation
This pull request implements a feature or fix, so it must include a changelog entry. See CONTRIBUTING.md#changelog for instructions. |
4db3814
to
a390ea1
Compare
@@ -350,6 +350,10 @@ export const updateInlineLockKey = 'CODEWHISPERER_INLINE_UPDATE_LOCK_KEY' | |||
export const newCustomizationMessage = 'You have access to new Amazon Q customizations.' | |||
|
|||
// Start of QCT Strings | |||
|
|||
// feature flag for SQL transformations | |||
export const isSQLTransformReady = true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will update to false
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For future reference: feature flags should live in dev/config.ts
, or as an "experiment" https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#preview-releases-and-experiments
c643ffa
to
e12ed81
Compare
packages/core/src/amazonqGumby/chat/controller/messenger/messenger.ts
Outdated
Show resolved
Hide resolved
Not ready to be merged yet (as of 10/31 morning) |
|
Update: ready to be merged (feature is behind a feature flag) |
QCT wants to support converting embedded SQL in Java apps. Allow users to start a transformation for SQL conversions. --- <!--- REMINDER: Ensure that your PR meets the guidelines in CONTRIBUTING.md --> License: I confirm that my contribution is made under the terms of the Apache 2.0 license. --------- Co-authored-by: David Hasani <[email protected]>
QCT wants to support converting embedded SQL in Java apps. Allow users to start a transformation for SQL conversions. --- <!--- REMINDER: Ensure that your PR meets the guidelines in CONTRIBUTING.md --> License: I confirm that my contribution is made under the terms of the Apache 2.0 license. --------- Co-authored-by: David Hasani <[email protected]>
QCT wants to support converting embedded SQL in Java apps. Allow users to start a transformation for SQL conversions. --- <!--- REMINDER: Ensure that your PR meets the guidelines in CONTRIBUTING.md --> License: I confirm that my contribution is made under the terms of the Apache 2.0 license. --------- Co-authored-by: David Hasani <[email protected]>
QCT wants to support converting embedded SQL in Java apps. Allow users to start a transformation for SQL conversions. --- <!--- REMINDER: Ensure that your PR meets the guidelines in CONTRIBUTING.md --> License: I confirm that my contribution is made under the terms of the Apache 2.0 license. --------- Co-authored-by: David Hasani <[email protected]>
QCT wants to support converting embedded SQL in Java apps. Allow users to start a transformation for SQL conversions. --- <!--- REMINDER: Ensure that your PR meets the guidelines in CONTRIBUTING.md --> License: I confirm that my contribution is made under the terms of the Apache 2.0 license. --------- Co-authored-by: David Hasani <[email protected]>
QCT wants to support converting embedded SQL in Java apps. Allow users to start a transformation for SQL conversions. --- <!--- REMINDER: Ensure that your PR meets the guidelines in CONTRIBUTING.md --> License: I confirm that my contribution is made under the terms of the Apache 2.0 license. --------- Co-authored-by: David Hasani <[email protected]>
Problem
QCT wants to support converting embedded SQL in Java apps.
Solution
Allow users to start a transformation for SQL conversions.
License: I confirm that my contribution is made under the terms of the Apache 2.0 license.