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

"bookmark" posts require a title/content #58

Closed
cweiske opened this issue May 31, 2017 · 6 comments
Closed

"bookmark" posts require a title/content #58

cweiske opened this issue May 31, 2017 · 6 comments

Comments

@cweiske
Copy link
Contributor

cweiske commented May 31, 2017

My tt-rss-micropub plugin creates bookmarks with only the bookmark-of property set. The wordpress micropub endpoint then returns an error, declaring that

Content, title, and excerpt are empty.

We had this for "like" posts already in #41. Is there a reason that bookmark posts do not work with the URL only?


(Followup to https://github.com/cweiske/tt-rss-micropub/issues/1)

@snarfed
Copy link
Member

snarfed commented May 31, 2017

hey, thanks for filing, and sorry for the trouble! cc @miklb. i can definitely add bookmark-of support.

i'm not sure if i have a more general fix, though. wordpress does indeed require at least one of content, title, or excerpt, but as you note, many micropub posts omit those, and depend on the server to render appropriately. i don't yet have a good way to render arbitrary mf2 data, so i may need to just add support for individual properties like this as they come up.

@miklb
Copy link

miklb commented May 31, 2017

I think the combination of semantic linkbacks and post kinds would handle it if the tt-rss plugin passed those to the micropub endpoint.

At least from my brief testing with Quill that's been the case.

@snarfed
Copy link
Member

snarfed commented May 31, 2017

afaik semantic-linkbacks is just for incoming webmentions (replies, likes, etc), not posts.

@dshanske
Copy link
Member

Post Kinds tries to handle it, yes

@snarfed
Copy link
Member

snarfed commented May 31, 2017

broader discussion around this in pfefferle/SemPress#51 (comment)

@cweiske
Copy link
Contributor Author

cweiske commented May 31, 2017

Ok, so the main problem is that bookmarks are not supported at all :)

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

No branches or pull requests

4 participants