docs icon indicating copy to clipboard operation
docs copied to clipboard

feat: Update Highlights docs

Open BekahHW opened this issue 1 year ago • 1 comments

Current

We have "Fill content" and "Add and Summarize" buttons in the "Highlight suggestions" section that are not covered here.

Originally posted by @adiati98 in https://github.com/open-sauced/docs/pull/213#discussion_r1405228070

Solution

  • [ ] add directions on adding a highlight from the highlight suggestion.
  • [ ] Explain both the "Fill content" and "Add and Summarize" buttons
  • [ ] Remove the existing gif for the highlight
  • [ ] Create a new gif with one that shows the suggestions and name it "highlight.gif". (The naming is important so we don't break anything that's been linked elsewhere.)

BekahHW avatar Dec 06 '23 17:12 BekahHW

Hi can i take this up?

oluwatobi2001 avatar May 05 '24 01:05 oluwatobi2001

Hi can i take this up?

Hey @oluwatobi2001. Just type .take in the comment section and you'll be automatically assigned.

CBID2 avatar May 27 '24 17:05 CBID2

Thanks for taking this on! If you have not already, join the conversation in our Discord

github-actions[bot] avatar May 27 '24 17:05 github-actions[bot]

Thanks for taking this on! If you have not already, join the conversation in our Discord

github-actions[bot] avatar May 27 '24 17:05 github-actions[bot]

Hi @oluwatobi2001, are you still interested in working on this issue? If not, I would like to take it. Let me know. Thanks!

vivitt avatar Jun 05 '24 09:06 vivitt

Yes, I'm still interested.

On Wed, Jun 5, 2024 at 10:19 AM Viviana Yanez @.***> wrote:

Hi @oluwatobi2001 https://github.com/oluwatobi2001, are you still interested in working on this issue? If not, I would like to take it. Let me know. Thanks!

— Reply to this email directly, view it on GitHub https://github.com/open-sauced/docs/issues/224#issuecomment-2149303301, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGW7FFK6FZPFO5TSVBTL7S3ZF3JZLAVCNFSM6AAAAABAJXZWJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBZGMYDGMZQGE . You are receiving this because you were mentioned.Message ID: @.***>

oluwatobi2001 avatar Jun 05 '24 16:06 oluwatobi2001

Hi @oluwatobi2001,

Please read our contributing guidelines. If you want to work on this issue, please assign yourself to the issue. If this issue is still not assigned by next Monday, let's give it to someone else who's interested. 🙂

adiati98 avatar Jun 06 '24 07:06 adiati98

Thanks @adiati98, I would love to take on this one :)

vivitt avatar Jun 12 '24 12:06 vivitt

.take

vivitt avatar Jun 12 '24 12:06 vivitt

Thanks for taking this on! If you have not already, join the conversation in our Community

github-actions[bot] avatar Jun 12 '24 12:06 github-actions[bot]

Hello! I have a couple of questions before submitting a PR for this issue:

First, I understand that this change should prioritize the Highlight suggestions section usage, so the current indications should remain in the docs but maybe below the new instructions and the new gif. Do you think this approach is correct, or does anyone have any other thoughts on this?

Second, I encountered an issue while using the "Add and summarize" button. As shown in the gif below, I receive an error message when I try to summarize a PR:
gif_01

I am also not able to use the Auto-Summarize feature if I paste the link manually: gif_02

So I wanted to check if I am following the right steps here or if I am missing something.

Thanks a lot for any help :)

vivitt avatar Jun 14 '24 10:06 vivitt

Hey @vivitt,

Sorry, I don't quite understand what do you mean in the first question. Can you clarify it for me?

As for your question about auto-summarize, I can confirm that I also have the same issue. You might want to create an issue on our app repo to report this if there is no similar issue available. 😊

adiati98 avatar Jun 14 '24 17:06 adiati98

Hello @adiati98 ! Didn't find a similar open issue, so I created one.

For my first question, I was confused because the How to Add a Highlight section now includes a set of bullet points and a GIF that need to be replaced. My bad for not explaining that clearly.

What I wanted to know is if should I document the "Fill Content" and "Add and Summarize" buttons by adding more bullet points below the existing ones, or is it better to reorganize the section by first documenting the "Highlight Suggestions" buttons and then the option of manually pasting a link?

Thanks!

vivitt avatar Jun 17 '24 08:06 vivitt

Hey @vivitt,

I think reorganizing would be better.

We can say something like, "There are two ways to add highlights, with the Highlight suggestions and manually." Then, follow the checkbox on the issue.

The instructions are all in bullet points for better read and understanding.

You can draft it first, and we'll see how we can improve and clarify them. And feel free to let me know if you have further questions or need help. ✨

adiati98 avatar Jun 19 '24 11:06 adiati98

Thanks @adiati98! Here's a possible draft for organizing the instructions:

## How to Add a Highlight

There are two ways to adding highlights:

### Using Highlight Suggestions

- Go to [OpenSauced's Highlights feed](https://app.opensauced.pizza/feed).
- Click the "Post a highlight to show your work!" input.
- Search the relevant blog post, pull request, or issue in the "Highlight suggestions" section.
- Either use the "Fill content" button to paste the suggested link or the "Add and Summarize" button to paste the link and Auto-Summarize the content.

![highlights demo](highlight.gif)

### Adding a Link Manually

- Go to [OpenSauced's Highlights feed](https://app.opensauced.pizza/feed).
- Click the "Post a highlight to show your work!" input.
- Paste the URL to your blog post, pull request, or issue in the bottom input.
- Either use our Auto-Summarize feature or write your own summary.

What do you think? Any suggestions? Thanks :)

vivitt avatar Jun 21 '24 13:06 vivitt

Hey @vivitt,

Sorry for taking a while to respond.

This looks great! ✨ However, I can see a room for improvement based on below things:

  • The first two bullet points in each method are the same. We might be able to avoid repetition here.
  • The "Highlight suggestions" only suggest issue and PRs, not blog posts.

Below is a raw suggestion. Feel free to give other ideas.

## How to Add a Highlight

First, go to [OpenSauced's Highlights feed](https://app.opensauced.pizza/feed), then click the "Post a highlight to show your work!" input.

There are two ways to adding highlights:

### 1. Using Highlight Suggestions

- Search the pull request or issue that you want to add in the "Highlight suggestions" section.
- Click the "Fill content" button to paste the suggested link or the "Add and Summarize" button to paste the link and auto-summarize the content.

:::note

The "Highlight suggestions" are only suggesting issues and pull requests that you created. For blog posts, you need to add them manually.

:::

![highlights demo](highlight.gif)

### 2. Adding a Link Manually

- Paste the URL to your blog post, pull request, or issue in the bottom input.
- Either use our Auto-Summarize feature or write your own summary.

adiati98 avatar Jun 25 '24 20:06 adiati98