Closed teodorus-nathaniel closed 3 weeks ago
The changes in this pull request focus on modifying the GET
request handler in the src/routes/api/github/auth/profile/+server.ts
file. The update ensures that user data is returned in the response after fetching it from the GitHub API. The existing logic for handling access and refresh tokens remains unchanged, maintaining the control flow and error handling while enhancing the response to include user information.
File Path | Change Summary |
---|---|
src/routes/api/github/auth/profile/+server.ts | Modified the GET request handler to include user data in the JSON response after fetching it from GitHub. |
integrations/autoinvoicing/src/index.ts | Updated onIssueComment event specifications, changing the name and filter action. |
src/lib/server/trigger-dev/jobs/check-run.ts | Added new utility functions for handling check run comments and updated the createJob function. |
src/lib/server/trigger-dev/jobs/index.ts | Defined a new job for handling issue comments triggered by the onIssueComment event. |
src/lib/server/trigger-dev/jobs/issues-comment.ts | Introduced a new function to process issue comment events from GitHub. |
src/lib/server/trigger-dev/jobs/pull-requests-review.ts | Added a function for reinserting comments associated with pull requests and updated job logic. |
src/lib/server/trigger-dev/utils.ts | Added new utility functions and types for managing pull requests and comments. |
src/lib/server/trigger-dev/jobs/issues-creation.ts | Updated the submissionHeaderComment function call to include a new prefix for issue comments. |
integrations/autoinvoicing/package.json | Incremented the version number from 0.0.9 to 0.0.10 . |
package.json | Updated the version for the @holdex/autoinvoicing dependency from 0.0.9 to 0.0.10 . |
Objective | Addressed | Explanation |
---|---|---|
Make message from the bot sticky to the bottom of the thread (#283) | ❌ | The changes do not relate to thread message behavior. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@teodorus-nathaniel is attempting to deploy a commit to the Holdex Accelerator Team on Vercel.
A member of the Team first needs to authorize it.
@teodorus-nathaniel
Thank you!.
Testing comment
Comment
Another comment
@coderabbitai review
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
autoinvoice | ❌ Failed (Inspect) | Nov 5, 2024 9:45am |
@teodorus-nathaniel nice one. We'll leave our review directly here.
@markholdex ⚠️⚠️⚠️ You must submit the time spent on this PR. ⚠️⚠️⚠️
@teodorus-nathaniel We need the app to use the new version of @holdex/autoinvoicing
@teodorus-nathaniel We need the app to use the new version of @holdex/autoinvoicing
Thanks for reminding, I forgot about that, I updated it in my latest commit e9f8c8a
@teodorus-nathaniel Let's push 1 more PR with updated package lock files
@teodorus-nathaniel Let's push 1 more PR with updated package lock files
Merged in
Resolves #283
FYI, updated
@holdex/autoinvoicing
package to enableissue_comment
event handling, so it needs to be republishedSummary by CodeRabbit
Summary by CodeRabbit
0.0.9
to0.0.10
.