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

Bug: summarize feature not working #231

Closed
2 tasks done
BekahHW opened this issue Jul 28, 2023 · 6 comments
Closed
2 tasks done

Bug: summarize feature not working #231

BekahHW opened this issue Jul 28, 2023 · 6 comments
Labels
🐛 bug Something isn't working 👀 needs triage

Comments

@BekahHW
Copy link
Member

BekahHW commented Jul 28, 2023

Describe the bug

The summarize feature returns an error instead of a summary.

Steps to reproduce

  1. go to a PR.
  2. Click post highlight
  3. click summarize.
  4. see this error

image

Browsers

Chrome

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@BekahHW BekahHW added 🐛 bug Something isn't working 👀 needs triage labels Jul 28, 2023
@diivi
Copy link
Contributor

diivi commented Jul 28, 2023

You mean the PR Description generator button right? It does work for me on beta, tried to follow the steps you gave.

@bdougie
Copy link
Member

bdougie commented Jul 28, 2023

This is most likely related to this pr #230.

The feature doesn't work without auth and there is not a proper error message.

At the very least we should update the error message to be more helpful.

@Anush008
Copy link
Member

The highlight summary generator seems to be working for me.
Untitled2

@BekahHW, could you share the link to the PR that generated the error for you?

@BekahHW
Copy link
Member Author

BekahHW commented Aug 7, 2023

Sorry, didn't have a computer with me last week. Seems like I get that error when I use the feature on a draft blog post.

image

@bdougie
Copy link
Member

bdougie commented Aug 8, 2023

That is helpful context. I think we start by making the error message return from the API a bit more descriptive.

But we also came across the draft PR issue before too. We can block drafts from being summarized.

@a0m0rajab
Copy link
Contributor

I could not reproduce this issue, do we still have this issue or did we solve it already?

@BekahHW BekahHW closed this as completed Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working 👀 needs triage
Projects
None yet
Development

No branches or pull requests

5 participants