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

[Feature request] Add option for "inbox" to take you to /notifications instead of /message/unread #11

Open
NSExceptional opened this issue Jul 15, 2022 · 7 comments

Comments

@NSExceptional
Copy link

Looks like Reddit is deprecating /message/unread for some users on New Reddit, and instead shows notifications over at /notifications now

@voidref
Copy link
Owner

voidref commented Jul 17, 2022

Is this happening for you?

I just updated the code ... a bunch, and it's not even letting me log in any more.

Check it out:

https://github.com/voidref/orangered/tree/bit-rot-work

Does this let you log in?

@voidref
Copy link
Owner

voidref commented Jul 18, 2022

Looks like OAuth needs to be implemented ... not sure I'm really motivated for that kind of effort.

PRs welcome.

@NSExceptional
Copy link
Author

Oof, me either. Scared to try a new version because of that. I might just hex edit the binary to open the new address instead. I'll make a new issue about OAuth and make this a dependent of it, if you want. We can label it Help Wanted and maybe someone will help out?

@voidref
Copy link
Owner

voidref commented Sep 5, 2022

Sounds like a plan, make it so!

@tinder-tannerbennett
Copy link

Done, #13. I can't add the "help wanted" label though, I think you have to.

So, is OAuth really a prerequisite for new features? Does updating the app log you out or something? Or could this still be done with an update without OAuth?

@tinder-tannerbennett
Copy link

Heh, sorry, I'm on my work computer. I'm @NSExceptional

@voidref
Copy link
Owner

voidref commented Sep 8, 2022

I don't think so, it should still work if you have the cached auth. You could always try just changing the path in the source and recompiling.

@voidref voidref added Help Wanted Pull Requests appreciated! and removed Help Wanted Pull Requests appreciated! labels Sep 8, 2022
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

3 participants