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 private syndication feeds via a user feedToken #3074

Merged
merged 3 commits into from
Nov 25, 2020

Conversation

rigelk
Copy link
Collaborator

@rigelk rigelk commented Aug 13, 2020

This PR adds a token for syndication feeds that need to stay private to the user. It allows users to add their subscriptions feeds to their feed aggregator.

New view to deal with access tokens:
Screenshot_2020-08-27 Applications - PeerTube

Button in the top-right corner of the subscription video list, to copy the RSS feed url:
Screenshot_2020-08-27 Subscriptions - PeerTube

TODO:

  • private syndication feed for subscriptions
  • route to get the feedToken
  • route to regenerate feedToken
  • API tests
  • view listing tokens and allowing to regenerate the token

@rigelk rigelk linked an issue Aug 14, 2020 that may be closed by this pull request
@rigelk rigelk added the Status: Requires Tests Either requires manual test, or writing tests, or both label Aug 16, 2020
@rigelk rigelk marked this pull request as ready for review August 16, 2020 13:22
@rigelk rigelk removed the Status: Requires Tests Either requires manual test, or writing tests, or both label Aug 26, 2020
@rigelk rigelk requested a review from Chocobozzz August 26, 2020 13:36
@rigelk rigelk force-pushed the token branch 2 times, most recently from 43b5c01 to aceae0c Compare August 27, 2020 08:17
@rigelk
Copy link
Collaborator Author

rigelk commented Nov 19, 2020

@Chocobozzz could you check if that is what you meant by usage of Observables?

@Chocobozzz Chocobozzz merged commit 18490b0 into Chocobozzz:develop Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RSS Feed for user subscriptions
2 participants