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

allow notification display and management #421

Open
yuvalif opened this issue Aug 15, 2024 · 5 comments
Open

allow notification display and management #421

yuvalif opened this issue Aug 15, 2024 · 5 comments

Comments

@yuvalif
Copy link

yuvalif commented Aug 15, 2024

this gh extension: https://github.com/meiji163/gh-notify
allows for commandline interaction with notifications.
would be useful if this could be integrated into gh-dash, so the display and interaction are done using TUI instead of commandline, and are fro mthe same tool

@joshmedeski
Copy link

I think it would be great to integrate GitHub notifications (in general) into gh-dash.

@dlvhdr let me know if you'd be down for me to do some work on this. I'd recommend we design the UI first.

@dlvhdr
Copy link
Owner

dlvhdr commented Sep 22, 2024

Of course I'm down, sounds great!

@RoseSecurity
Copy link

This is awesome! Any progress with the design?

@ccoVeille
Copy link

Thanks @RoseSecurity for reactivating this old discussion.

I wasn't aware gh-notify existed. I'll take a look at it. From what I can see it's a pure script. So it would be about reverse-engineering what the code does.

Maybe gh-not (hi @nobe4 👋), also in Go, could be a better candidate for anyone interested to bring notifications to gh-dash, or simply to use it directly.

https://github.com/nobe4/gh-not

@nobe4
Copy link

nobe4 commented Mar 13, 2025

The code behind gh-not is more than just a notification viewer.
So I don't know how much of this codebase would fit into gh-dash.

The main ideas of gh-not are:

  • Hardened notification fetching and caching.

    The /notification API was so flaky for me, it gave me the incentive to write that in the first place.
    There's a significant portion of the code dedicated only to pulling notifications and their context, and handling a local cache.

  • Configured actions, so I could for instance, mark all notifications from dependabot, as done automatically.

    rules:
      - filters:
          - .author.login == "dependabot[bot]"
        action: done
  • REPL to access the notifications in a TUI.

None of that code is currently exported, but if there's interest, I could make some of that available to others. 🤝

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants