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

Mimic needinfo behavior? #2

Open
pocmo opened this issue Jul 19, 2018 · 1 comment
Open

Mimic needinfo behavior? #2

pocmo opened this issue Jul 19, 2018 · 1 comment
Labels
feature Feature request P1

Comments

@pocmo
Copy link

pocmo commented Jul 19, 2018

I don't know how, but needinfo is one of the features I'm missing the most. GitHub notifications just go away once I read them. So here is my user story:

As an engineer that gets a gazillion GitHub notifications, that go away once read, I would like to have a more permanent way of marking something as "I need to get back to this".

@mcomella mcomella added the feature Feature request label Jul 28, 2018
@mcomella
Copy link
Owner

mcomella commented Aug 4, 2018

fwiw, some ways to handle this now:

I use my bugmail as my github notifications and, if there's something I need to get back to, I leave it unread. It doesn't solve the problem the same way because:

  • Unread emails will drop down my inbox
  • New emails in the thread will mask why I left it unread in the first place
  • Others can't force a NI on me so it's possible I'll miss their request for questions

fwiw, afaik Refined GitHub lets you mark GitHub notifications as unread directly in the GitHub interface (I haven't vetted most of their code and don't vouch for them :).

Another thing that I do is copy comment URLs I need to get back to into a note-taking program that I check often.

:hurtrealbad:


Initial thoughts on implementation: we host a server (Moz heroku?) that will store the pending NI state. We should probably whitelist our repositories to avoid excessive processing.

The add-on can communicate new NI requests to the server (either parsing new comments or injecting new user interface elements to do so) and can retrieve pending NI requests from the server. We probably should create a dashboard to see all pending NI requests for the current user.

We should also see if an existing solution exists for this.

@mcomella mcomella added the P1 label Sep 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature request P1
Projects
None yet
Development

No branches or pull requests

2 participants