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

feat: Updated to the new AI PR description endpoint #94

Merged
merged 8 commits into from
May 19, 2023

Conversation

Anush008
Copy link
Member

@Anush008 Anush008 commented May 19, 2023

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Description

  • This PR updates the AI PR description feature to use the new https://api.opensauced.pizza/v1/prs/description/generate endpoint.
  • The AI description page of the extension dropdown has been updated and the OpenAI token input has been removed.
  • The Description config types have been updated to reflect the new changes.
  • A new function
    export const insertTextAtCursor = async (textArea: HTMLTextAreaElement, text: string) => {

    has been added to emulate the typewriter effect from a string rather than a stream, since that's response from the API endpoint. The older function has been retained for future use.

Related Tickets & Documents

Resolves #85.

Mobile & Desktop Screenshots/Recordings

Untitled

Added tests?

  • πŸ‘ yes
  • πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.opensauced.pizza
  • πŸ• dev.to/opensauced
  • πŸ“• storybook
  • πŸ™… no documentation needed

@Anush008 Anush008 requested review from diivi and bdougie May 19, 2023 13:32
@Anush008 Anush008 changed the title New description endpoint feat: New description endpoint May 19, 2023
@Anush008 Anush008 changed the title feat: New description endpoint feat: Updated to the new AI PR description endpoint May 19, 2023
@bdougie
Copy link
Member

bdougie commented May 19, 2023

I prefer "generated by OpenSauced"

"Content" is already vague and at least we can allure the reader to click the link.

Copy link
Member

@bdougie bdougie left a comment

Choose a reason for hiding this comment

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

Can we turn (power button) on the AI feature by default. We are hiding the feature by default and we should let them turn it off if needed.

Note: not for this PR, but we should allow users to toggle off other features like the "upvote button"

src/utils/aiprdescription/cursorPositionInsert.ts Outdated Show resolved Hide resolved
src/utils/aiprdescription/cursorPositionInsert.ts Outdated Show resolved Hide resolved
@Anush008
Copy link
Member Author

Can we turn (power button) on the AI feature by default. We are hiding the feature by default and we should let them turn it off if needed.

Done. bc0f4a9.

@bdougie bdougie merged commit cf8f64c into beta May 19, 2023
github-actions bot pushed a commit that referenced this pull request May 19, 2023
## [1.1.0-beta.7](v1.1.0-beta.6...v1.1.0-beta.7) (2023-05-19)

### πŸ• Features

* Updated to the new AI PR description endpoint ([#94](#94)) ([cf8f64c](cf8f64c))
@github-actions
Copy link

πŸŽ‰ This PR is included in version 1.1.0-beta.7 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

@Anush008 Anush008 deleted the new-description-endpoint branch May 19, 2023 23:47
github-actions bot pushed a commit that referenced this pull request May 20, 2023
## [1.1.0](v1.0.0...v1.1.0) (2023-05-20)

### πŸ“ Documentation

* how we create extension releases ([#81](#81)) ([10b1c48](10b1c48))

### πŸ› Bug Fixes

* add extra line and italics ([#93](#93)) ([2cbd790](2cbd790))
* update name in README ([dba4734](dba4734))

### πŸ• Features

* add/remove repos to/from insights. ([#77](#77)) ([8e4a6ed](8e4a6ed))
* OpenSauced AI PR description  ([#79](#79)) ([0e38777](0e38777))
* replace open issues count with highlights count ([#76](#76)) ([0fb8149](0fb8149))
* Updated to the new AI PR description endpoint ([#94](#94)) ([cf8f64c](cf8f64c))
@github-actions
Copy link

πŸŽ‰ This PR is included in version 1.1.0 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

@Anush008 Anush008 mentioned this pull request May 24, 2023
19 tasks
zer0and1 added a commit to zer0and1/open-sauced.ai that referenced this pull request Jul 26, 2023
## [1.1.0-beta.7](open-sauced/ai@v1.1.0-beta.6...v1.1.0-beta.7) (2023-05-19)

### πŸ• Features

* Updated to the new AI PR description endpoint ([#94](open-sauced/ai#94)) ([cf8f64c](open-sauced/ai@cf8f64c))
zer0and1 pushed a commit to zer0and1/open-sauced.ai that referenced this pull request Jul 26, 2023
## [1.1.0](open-sauced/ai@v1.0.0...v1.1.0) (2023-05-20)

### πŸ“ Documentation

* how we create extension releases ([#81](open-sauced/ai#81)) ([10b1c48](open-sauced/ai@10b1c48))

### πŸ› Bug Fixes

* add extra line and italics ([#93](open-sauced/ai#93)) ([2cbd790](open-sauced/ai@2cbd790))
* update name in README ([dba4734](open-sauced/ai@dba4734))

### πŸ• Features

* add/remove repos to/from insights. ([#77](open-sauced/ai#77)) ([8e4a6ed](open-sauced/ai@8e4a6ed))
* OpenSauced AI PR description  ([#79](open-sauced/ai#79)) ([0e38777](open-sauced/ai@0e38777))
* replace open issues count with highlights count ([#76](open-sauced/ai#76)) ([0fb8149](open-sauced/ai@0fb8149))
* Updated to the new AI PR description endpoint ([#94](open-sauced/ai#94)) ([cf8f64c](open-sauced/ai@cf8f64c))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Remove dependency on user provide openai tokens
3 participants