Skip to content
This repository has been archived by the owner on Aug 28, 2023. It is now read-only.

Merge upstream #5

Open
dshanske opened this issue Feb 10, 2022 · 3 comments
Open

Merge upstream #5

dshanske opened this issue Feb 10, 2022 · 3 comments

Comments

@dshanske
Copy link

Have you considered submitting a PR on this to Yarns?

@janboddez
Copy link
Owner

I hadn't (till now), but I'm also not against it. :)

While at the moment this is a separate plugin, it wouldn't take too much to merge the changes into Yarns itself. Way it works pretty much matches with jackjamieson2/yarns-microsub-server#100 (comment).

Only thing I didn't take into account (or wasn't aware of, perhaps) is Parse This's OPML parser. (At first glance, "mine," which is actually WordPress Core's with very minor changes, seems to do just a little bit more.)

Don't actively use Yarns, though (and while everything should function, there's a few things, like the ability to store source names and formats, that when addressed would allow for greatly improved OPML exports).

@janboddez
Copy link
Owner

I mean, if it helps, sure, I can try.

@dshanske
Copy link
Author

I'd wait... there's a long term plan to update Yarns I work on periodically.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants