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
Open

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

tantek opened this issue Mar 15, 2018 · 10 comments

Comments

@tantek
Copy link
Member

tantek 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
Copy link
Member

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
Copy link
Member

gRegorLove 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
Copy link
Member Author

tantek 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
Copy link
Member

Zegnat commented Mar 16, 2018

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

@gRegorLove
Copy link
Member

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

@kartikprabhu
Copy link
Member

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

@gRegorLove
Copy link
Member

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

@aaronpk: thoughts on updating it to prefer content?

@aaronpk
Copy link
Member

aaronpk 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
Copy link
Member

@dshanske
Copy link
Member

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
Labels
None yet
6 participants