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

Replace h-event 'description' with 'content' property #3

Open
tantek opened this issue Mar 15, 2018 · 10 comments

Comments

Projects
None yet
6 participants
@tantek
Copy link
Member

commented Mar 15, 2018

Splitting off from issue 2: it appears most (nearly all?) currently published h-event posts are using "content" (adopted from h-entry) instead of "description" (from mf1 hCalendar).

Should we drop "p-description" and replace it with "e-content" in h-event? (before transitioning h-event from Draft to Specification)

Are there any h-event (mf2, not backcompat mf1) consuming applications that depend on 'descripton'?

Are there any h-event (mf2, not backcompat mf1) publishers that only publish 'description' and not 'content'?

cc: @kevinmarks @Zegnat @aaronpk @martymcguire

(Originally published at: http://tantek.com/2018/074/b1/replace-event-description-with-content)

@kartikprabhu

This comment has been minimized.

Copy link
Member

commented Mar 15, 2018

does this also change backcompat for h-event? http://microformats.org/wiki/h-event#Parser_Compatibility currently mf1 description has no mf2 equivalent. mf2py uses p-description

@gRegorLove

This comment has been minimized.

Copy link
Member

commented Mar 15, 2018

My mf2 to iCalendar library consumes description. I use it on my site and a couple of external sites. I could update it pretty easily to prefer content if it exists, falling back to description.

I currently publish only h-event description property, not content. Same for those external sites.

@tantek

This comment has been minimized.

Copy link
Member Author

commented Mar 16, 2018

https://github.com/kartikprabhu Yes this also changes backcompat for hCalendar, 'description' would be interpreted as 'p-content' for h-event.

https://github.com/gRegorLove yes try updating the consuming code to prefer 'content' over 'description' and see if everything still works. We may have to deprecate (rather than completely remove) 'p-description'. See how the consuming code update goes, and then see how much trouble it would be to update to publishing 'content' instead of 'description'.

(Originally published at: http://tantek.com/2018/074/t1/)

@Zegnat

This comment has been minimized.

Copy link
Member

commented Mar 16, 2018

I wonder what this looks like in WordPress land? Ping @dshanske?

@gRegorLove

This comment has been minimized.

Copy link
Member

commented Mar 30, 2018

mf2 to iCalendar 0.0.2 now prefers content over description. Deployed on my site and working well so far.

@kartikprabhu

This comment has been minimized.

Copy link
Member

commented Mar 30, 2018

OK maybe the backcompat rules for h-event should be updated to convert description to p-content p-description

@gRegorLove

This comment has been minimized.

Copy link
Member

commented Mar 30, 2018

https://github.com/aaronpk/XRay appears to parse description and summary but not content.

@aaronpk: thoughts on updating it to prefer content?

@aaronpk

This comment has been minimized.

Copy link
Member

commented Mar 30, 2018

I'd be happy to update it! Pretty sure I just followed the spec when deciding which to look for in the first place

@gRegorLove

This comment has been minimized.

Copy link
Member

commented Mar 30, 2018

@dshanske

This comment has been minimized.

Copy link

commented Jun 26, 2019

Didn't notice this. I only publish h-events from Micropub, not native UI right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.