glance icon indicating copy to clipboard operation
glance copied to clipboard

feat: new optional OAuth2 configuration for the Reddit widget

Open s0ders opened this issue 8 months ago • 2 comments

This PR brings the following:

  • New reddit-app-name, reddit-client-id and reddit-client-secret optional configurations for the Reddit widget to authenticate requests sent to the Reddit API (avoiding being blocked when Glance is self-hosted)
  • Refactoring the of fetchSubredditPosts function which is now a method of redditWidget since it's unexported and only uses parameters which are fields of the redditWidget struct.
  • Updated documentation regarding the new Reddit widget parameters.

ref: #509

s0ders avatar Mar 28 '25 10:03 s0ders

Eagerly waiting for this one to be merged! can't view my subreddit updates without it :(

Nirzak avatar Mar 30 '25 14:03 Nirzak

@svilenmarkov please review this. Thanks

Nirzak avatar Apr 10 '25 21:04 Nirzak

I wasn't able to make changes to the PR since you've disabled edit access for maintainers, so I used your changes, made some tweaks and committed them to the dev branch, adding you as the co-author. Thanks for contributing!

svilenmarkov avatar Apr 28 '25 17:04 svilenmarkov

My bad, I didn't check that I disabled edit access. Thanks!

s0ders avatar Apr 30 '25 10:04 s0ders