Skip to content

feat(ai-scan-support): Feedback mechanism and Copy Failure Details button (only for HTML reports)#7576

Merged
v-viyada merged 14 commits intomicrosoft:mainfrom
ashkan0x:users/ashkan0x/feedback-and-copy-details
Apr 22, 2025
Merged

feat(ai-scan-support): Feedback mechanism and Copy Failure Details button (only for HTML reports)#7576
v-viyada merged 14 commits intomicrosoft:mainfrom
ashkan0x:users/ashkan0x/feedback-and-copy-details

Conversation

@ashkan0x
Copy link
Copy Markdown
Member

This PR follows: #7570

Details

  • Feedback mechanism is enabled when feedbackURL is provided AND the issue's guidance matches AI_SCAN
  • Copy failure details button is implemented for HTML reports
feat.ai-scan-support.mp4
Motivation
Context

Pull request checklist

  • Addresses an existing issue: #0000
  • Ran yarn fastpass
  • Added/updated relevant unit test(s) (and ran yarn test)
  • Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
  • PR title AND final merge commit title both start with a semantic tag (fix:, chore:, feat(feature-name):, refactor:). See CONTRIBUTING.md.
  • (UI changes only) Added screenshots/GIFs to description above
  • (UI changes only) Verified usability with NVDA/JAWS

…+ implement copy failure details button in HTML reports
@ashkan0x ashkan0x requested a review from a team as a code owner April 10, 2025 13:24
Copy link
Copy Markdown
Contributor

@homitc homitc left a comment

Choose a reason for hiding this comment

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

Looking good! I think with some more test cases this will be good to go!

Copy link
Copy Markdown
Contributor

@homitc homitc left a comment

Choose a reason for hiding this comment

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

Thanks so much for all this work! It looks great!

@v-viyada v-viyada merged commit e63cd28 into microsoft:main Apr 22, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants