-
-
Notifications
You must be signed in to change notification settings - Fork 30
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: highlight redirection, multiple AI generation, draft commit messages #163
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
## [1.4.1-beta.1](v1.4.0...v1.4.1-beta.1) (2023-06-04) ### π Bug Fixes * highlight redirection, multiple AI generation, draft commit messages ([#163](#163)) ([03b91ab](03b91ab))
π This PR is included in version 1.4.1-beta.1 π The release is available on GitHub release Your semantic-release bot π¦π |
Thanks for this, but ideally we should wait for a little more time before someone comes and picks up simple and low priority good first issues (2 of them linked here). |
Also please assign these tasks to yourself first. |
## [1.5.0](v1.4.0...v1.5.0) (2023-06-06) ### π§βπ» Code Refactoring * removed post highlight message passing ([#165](#165)) ([bad2112](bad2112)) ### π Features * extension settings ([#132](#132)) ([722cbbe](722cbbe)) ### π Bug Fixes * highlight redirection, multiple AI generation, draft commit messages ([#163](#163)) ([03b91ab](03b91ab)) * post-highlight-page-check ([#164](#164)) ([ed98eb2](ed98eb2)) * remove configuration enabled check from AI button ([#170](#170)) ([cedca79](cedca79))
π This PR is included in version 1.5.0 π The release is available on GitHub release Your semantic-release bot π¦π |
## [1.4.1-beta.1](open-sauced/ai@v1.4.0...v1.4.1-beta.1) (2023-06-04) ### π Bug Fixes * highlight redirection, multiple AI generation, draft commit messages ([#163](open-sauced/ai#163)) ([03b91ab](open-sauced/ai@03b91ab))
## [1.5.0](open-sauced/ai@v1.4.0...v1.5.0) (2023-06-06) ### π§βπ» Code Refactoring * removed post highlight message passing ([#165](open-sauced/ai#165)) ([bad2112](open-sauced/ai@bad2112)) ### π Features * extension settings ([#132](open-sauced/ai#132)) ([722cbbe](open-sauced/ai@722cbbe)) ### π Bug Fixes * highlight redirection, multiple AI generation, draft commit messages ([#163](open-sauced/ai#163)) ([03b91ab](open-sauced/ai@03b91ab)) * post-highlight-page-check ([#164](open-sauced/ai#164)) ([ed98eb2](open-sauced/ai@ed98eb2)) * remove configuration enabled check from AI button ([#170](open-sauced/ai#170)) ([cedca79](open-sauced/ai@cedca79))
What type of PR is this? (check all applicable)
Description
Resolves Bug: Getting PR commit messages is unstable Β #131 by passing the PR URL without any query params or IDs. With this update, appending
/commits
to the endpoint, the current inability to get commit messages of draft PRs mentioned at feat: OpenSauced AI PR description Β #79 (comment) will be resolved.ai/src/utils/fetchGithubAPIData.ts
Lines 23 to 24 in b697b70
Resolves Bug: post to highlight in dev mode is redirecting to prod websiteΒ #153 by updating
href
to the configured domain. b6f1dd0.Resolves Bug: AI suggestions UXΒ #119 by disabling the button events when the AI generation is underway. fc26ef8.
Additionally, refactors the latest highlight feat, moves the
Highlights
type definition to the types file for reusability, usescachedFetch()
for getting the highlights. b2ec143Related Tickets & Documents
closes #131
closes #153
closes #119
Mobile & Desktop Screenshots/Recordings
Added tests?
Added to documentation?