-
Notifications
You must be signed in to change notification settings - Fork 243
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): Implementing the feedback system for internal users to improve unit test generation quality. #5465
Conversation
...jetbrains-community/src/software/aws/toolkits/jetbrains/ui/feedback/TestGenFeedbackDialog.kt
Dismissed
Show dismissed
Hide dismissed
...jetbrains-community/src/software/aws/toolkits/jetbrains/ui/feedback/TestGenFeedbackDialog.kt
Dismissed
Show dismissed
Hide dismissed
...jetbrains-community/src/software/aws/toolkits/jetbrains/ui/feedback/TestGenFeedbackDialog.kt
Dismissed
Show dismissed
Hide dismissed
...jetbrains-community/src/software/aws/toolkits/jetbrains/ui/feedback/TestGenFeedbackDialog.kt
Fixed
Show fixed
Hide fixed
...jetbrains-community/src/software/aws/toolkits/jetbrains/ui/feedback/TestGenFeedbackDialog.kt
Fixed
Show fixed
Hide fixed
...brains-community/src/software/aws/toolkits/jetbrains/ui/feedback/FeatureDevFeedbackDialog.kt
Dismissed
Show dismissed
Hide dismissed
...jetbrains-community/src/software/aws/toolkits/jetbrains/ui/feedback/TestGenFeedbackDialog.kt
Dismissed
Show dismissed
Hide dismissed
...jetbrains-community/src/software/aws/toolkits/jetbrains/ui/feedback/TestGenFeedbackDialog.kt
Dismissed
Show dismissed
Hide dismissed
@@ -16,6 +16,6 @@ class FeatureDevFeedbackDialog(project: Project) : FeedbackDialog(project) { | |||
} | |||
|
|||
init { | |||
title = message("feedback.title.amazonq.feature_dev") | |||
title = message("feedback.title.amazonq.send_feedback") |
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.
Generalizing string name to reuse this for both /test and /dev.
...oftware/aws/toolkits/jetbrains/services/amazonqCodeTest/controller/CodeTestChatController.kt
Dismissed
Show dismissed
Hide dismissed
@@ -559,12 +560,25 @@ class CodeWhispererUTGChatManager(val project: Project, private val cs: Coroutin | |||
e is JsonParseException -> message("testgen.error.generic_technical_error_message") | |||
else -> message("testgen.error.generic_error_message") | |||
} | |||
|
|||
val buttonList = mutableListOf<Button>() | |||
if (isInternalUser(getStartUrl(project))) { |
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.
i think we need a discussion on why this is proliferating
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.
This PR is aimed for only internal users to get feedback about quality of UTG. So, I used this, but LMK if there is a better way todo this!
Thank you.
Problem
Solution
JB.PR.F.b.mov
Checklist
License
I confirm that my contribution is made under the terms of the Apache 2.0 license.