Skip to content
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

fix(ai-help): add short and extended explanatory guidance #9215

Merged
merged 1 commit into from
Jul 2, 2023

Conversation

LeoMcA
Copy link
Member

@LeoMcA LeoMcA commented Jul 2, 2023

Summary

#9214
#9208

Problem

We don't explain that outputs from AI Help are not guaranteed to be accurate, and that the links to the docs used to generate the prompt exist to validate the response.

Solution

Add a short explanation of this, which expands into a modal containing a longer one.


Screenshots

Before

Screen Shot 2023-07-02 at 16 04 36

After

Screen Shot 2023-07-02 at 15 47 50

Screen Shot 2023-07-02 at 15 47 54


How did you test this change?

yarn dev, tested across different screen sizes - did not test with responses in the chat, I guess our testing creds changed, so could potentially mess with autoscroll being a bit off, but I reckon this is ok for now

@LeoMcA LeoMcA requested a review from fiji-flo July 2, 2023 15:07
@github-actions github-actions bot added the plus work around features related to MDN Plus label Jul 2, 2023
Copy link
Contributor

@fiji-flo fiji-flo left a comment

Choose a reason for hiding this comment

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

Thanks for doing this on a Sunday. Looking good 👍

@LeoMcA LeoMcA merged commit 1da1d4e into main Jul 2, 2023
@LeoMcA LeoMcA deleted the ai-help-guidance branch July 2, 2023 15:19
@MrPetovan
Copy link

The problem of this features wasn't the lack of explanation about the potential inaccuracy, the problem is the inaccuracy itself.

You're operating a reference documentation, and you rolled out a feature encouraging to "cross-verify" the information this feature produces with your own information? This doesn't make any sense.

@oscherler
Copy link

I’m glad it’s about a web site and not a zoo. Imagine a lion escaping its cage, and they just put up a sign at the entrance saying “Caution, loose lion on the premises.” instead of closing the zoo until the lion is captured.

Can you please update the PR title to remove the word “fix,” and use something more accurate like “temporary mitigation?”

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plus:ai-help plus work around features related to MDN Plus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants