Support for Syndication Targets via q=syndicate-to #17

Closed
miklb opened this Issue Dec 6, 2016 · 5 comments

Comments

Projects
None yet
2 participants
@miklb

miklb commented Dec 6, 2016

As started in 2254d63, support the q=syndicate-to per the Micropub spec.

Granted, I'm not 100% sure how that differs from current implementations. Thanks for the considerations and I look forward to testing more.

@dissolve

This comment has been minimized.

Show comment
Hide comment
@dissolve

dissolve Dec 12, 2016

Collaborator

Code to fix this has been pushed to the live site. Can verify you are using the new version of the code when the settings page shows version information at the bottom.

Collaborator

dissolve commented Dec 12, 2016

Code to fix this has been pushed to the live site. Can verify you are using the new version of the code when the settings page shows version information at the bottom.

dissolve added a commit that referenced this issue Dec 12, 2016

issue #17: support syndicate-to query
This should resolve the issue but will wait for confirmation for @miklb
before closing
@miklb

This comment has been minimized.

Show comment
Hide comment
@miklb

miklb Dec 12, 2016

Just tested, and see version 1.1.4 on settings page and a button to force refresh Syndication Targets. However I'm still confused on how that is supposed to work. Not seeing options on the publish note page. Appreciate the work thus far on this.

miklb commented Dec 12, 2016

Just tested, and see version 1.1.4 on settings page and a button to force refresh Syndication Targets. However I'm still confused on how that is supposed to work. Not seeing options on the publish note page. Appreciate the work thus far on this.

@dissolve

This comment has been minimized.

Show comment
Hide comment
@dissolve

dissolve Dec 12, 2016

Collaborator

When you force refresh, is it getting a bulleted list of your endpoints?
You should see a multiselect field as the last item on the create page with those options.

Its expecting json only, and they have to have 'uid' and 'name' set (like https://www.w3.org/TR/micropub/#syndication-targets )
or they won't show up at all.

Here are samples of what it looks like for me.

https://ben.thatmustbe.me/upload/files/117_inkstone-post.png
https://ben.thatmustbe.me/upload/files/118_inkstone-settings.png

Collaborator

dissolve commented Dec 12, 2016

When you force refresh, is it getting a bulleted list of your endpoints?
You should see a multiselect field as the last item on the create page with those options.

Its expecting json only, and they have to have 'uid' and 'name' set (like https://www.w3.org/TR/micropub/#syndication-targets )
or they won't show up at all.

Here are samples of what it looks like for me.

https://ben.thatmustbe.me/upload/files/117_inkstone-post.png
https://ben.thatmustbe.me/upload/files/118_inkstone-settings.png

@miklb

This comment has been minimized.

Show comment
Hide comment
@miklb

miklb Dec 12, 2016

Determined issue was with the micropub endpoint I was using was sending application/x-www-form-urlencoded. Issue was opened to change behavior to send application/json, however has been fixed in Inkstone.

Closing as resolved. Thanks again for the help debugging.

miklb commented Dec 12, 2016

Determined issue was with the micropub endpoint I was using was sending application/x-www-form-urlencoded. Issue was opened to change behavior to send application/json, however has been fixed in Inkstone.

Closing as resolved. Thanks again for the help debugging.

@miklb miklb closed this Dec 12, 2016

@dissolve

This comment has been minimized.

Show comment
Hide comment
@dissolve

dissolve Dec 12, 2016

Collaborator

to be clear, inkstone was not requesting application/json specifically. this has been fixed as of v 1.1.6

Collaborator

dissolve commented Dec 12, 2016

to be clear, inkstone was not requesting application/json specifically. this has been fixed as of v 1.1.6

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