If a notification was related to a pull request the link we added to the item didn't work. This should now be fixed, so that when a user clicks on the item he is redirected to the correct pull request which is associated with the item.
This commit also removes an unnecessary "console.log" statement.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
## 📋 Pull Request Information
**Original PR:** https://github.com/feeddeck/feeddeck/pull/55
**Author:** [@ricoberger](https://github.com/ricoberger)
**Created:** 10/29/2023
**Status:** ✅ Merged
**Merged:** 10/29/2023
**Merged by:** [@ricoberger](https://github.com/ricoberger)
**Base:** `main` ← **Head:** `github-fix-notification-link-for-prs`
---
### 📝 Commits (1)
- [`d9c5728`](https://github.com/feeddeck/feeddeck/commit/d9c5728e69ae065367b90bdce366c0dabafe977b) [github] Fix Notification Links for PRs
### 📊 Changes
**2 files changed** (+20 additions, -7 deletions)
<details>
<summary>View changed files</summary>
📝 `supabase/functions/_shared/feed/github.ts` (+20 -6)
📝 `supabase/functions/_shared/feed/x.ts` (+0 -1)
</details>
### 📄 Description
If a notification was related to a pull request the link we added to the item didn't work. This should now be fixed, so that when a user clicks on the item he is redirected to the correct pull request which is associated with the item.
This commit also removes an unnecessary "console.log" statement.
<!--
Keep PR title verbose enough and add prefix telling about what source it touches e.g "[rss] Add feature xyz" or if the
the PR is not realated to a source use "[core]", e.g. "[core] Fix xyz".
If you add a breaking change within your PR you should add ":warning:" to the title,
e.g. ":warning: [core] My breaking change"
-->
<!--
Description of what have been changed. Please also reference an issue, when available.
-->
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/feeddeck/feeddeck/pull/55
Author: @ricoberger
Created: 10/29/2023
Status: ✅ Merged
Merged: 10/29/2023
Merged by: @ricoberger
Base:
main← Head:github-fix-notification-link-for-prs📝 Commits (1)
d9c5728[github] Fix Notification Links for PRs📊 Changes
2 files changed (+20 additions, -7 deletions)
View changed files
📝
supabase/functions/_shared/feed/github.ts(+20 -6)📝
supabase/functions/_shared/feed/x.ts(+0 -1)📄 Description
If a notification was related to a pull request the link we added to the item didn't work. This should now be fixed, so that when a user clicks on the item he is redirected to the correct pull request which is associated with the item.
This commit also removes an unnecessary "console.log" statement.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.