Closed antonengelhardt closed 5 months ago
👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labled with Status: Up for grabs
. You & others like you are the reason all of this works! So thank you & happy coding! 🚀
That is out of scope for this package. We only provide wrappers around the GitHub API
Feel free to submit this as a feature request to GitHub for their API. https://github.com/orgs/community/discussions/categories/api-and-webhooks
Describe the need
Hello,
the GitHub Raycast Extension has a command (
Notifications
) to fetch all notifications for the user using thelistNotificationsForAuthenticatedUser()
endpoint:When setting the
all
-parameter totrue
, it returns all notifications (including done ones). When setting it tofalse
, only unread notifications are returned.The GitHub Website groups notifications depending if they're done or undone. Additionally, a filter can display only the unread notifications:
@tom-sherman proposed to follow the GitHub style in a recently opened issue. Following the Github style would mean that the extension would have to group the notifications by undone/done and highlight unread ones, which is currently impossible, because there is no way to tell if a notification is done or not.
My proposal is that either
done
attribute ofboolean
type (then we can partition the notifications ourselves) orlistNotificationsForAuthenticatedUser()
endpoint gets more filters e.gshowDone
(bool
),showRead
(boolean
).Thanks for considering. Cheers.
SDK Version
v3.1.2
API Version
v20
Relevant log output
Code of Conduct