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.

Sign up for GitHub

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

Jump to bottom

Added a new OPML import option to import subscriptions from an url #482

Merged
merged 2 commits into from
Oct 25, 2022
Merged

Added a new OPML import option to import subscriptions from an url #482

merged 2 commits into from
Oct 25, 2022

Conversation

VGJohn
Copy link
Contributor

@VGJohn VGJohn commented Oct 25, 2022

Added a new OPML import option to import subscriptions from an url

Fixes #480

Checklist

  • Should this change be included in the release notes? If yes, please add a line in CHANGELOG.md
  • Have you tested in landscape?
  • Have you tested accessibility with TalkBack?
  • Have you tested in different themes?
  • Does the change work with a large display font?
  • Are all the strings localized?
  • Could you have written any new tests?
  • Did you include Compose previews with any components?
importbyurldemo.mp4

@VGJohn VGJohn requested a review from a team as a code owner October 25, 2022 03:21
geekygecko
geekygecko approved these changes Oct 25, 2022
View reviewed changes
Copy link
Member

@geekygecko geekygecko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, that was quick! You can code faster than I can add issues. Thanks again for your help. 🙏

@geekygecko geekygecko merged commit 10ba56c into Automattic:main Oct 25, 2022
@geekygecko geekygecko mentioned this pull request Oct 25, 2022
Closed
@VGJohn VGJohn deleted the import-by-url branch October 25, 2022 04:10
@scripting
Copy link

This is good news. 😄

If you want some OPML to test with, here's my list of podcast subscriptions.

http://feedland.org/opml?screenname=davewiner&catname=podcasts

@VGJohn
Copy link
Contributor Author

VGJohn commented Oct 25, 2022

Cheers for the extra OPML to test with, seems to work ok from what I can see 😄

opmlurltest.mp4

@scripting
Copy link

NICE!!! 💥

@ashiagr ashiagr added this to the 7.26 milestone Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@geekygecko geekygecko geekygecko approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
7.26
Development

Successfully merging this pull request may close these issues.

Import OPML from a URL
4 participants
@VGJohn @scripting @geekygecko @ashiagr