#wordpress 2017-07-08

2017-07-08 UTC
[kevinmarks] and davidmead joined the channel
#
davidmead
!tell pfefferle: I’ve added my child theme to GitHub - https://github.com/davidmead/child-themes
#
Loqi
Ok, I'll tell them that when I see them next
#
Loqi
[davidmead] child-themes: Wordpress child themes
[miklb] joined the channel
#
www.boffosocko.com
edited /WordPress (+85) "/* Getting Help */ #indieweb-wordpress channel"
(view diff)
tantek joined the channel
#
www.boffosocko.com
edited /WordPress (+1454) "Richard MacManus' indiewebifying post; general advice about indiewebifying"
(view diff)
[cleverdevil], prtksxna, prtksxna_, davidmead and [miklb] joined the channel
#
[miklb]
I appreciate all the work being done by the team working on the WP Coding Standards, but the amount of work being done has me questioning how accurate it was previously. Still waiting on support for phpcs 3.x
#
[miklb]
someone in the micro.blog wordpress channel is experiencing an issue where in the IndieWeb plugin, for other sites in the user profile, if you put more than 2 sites, even if you put them on separate lines, when you save the profile, it puts them on one line, which breaks it.
#
[miklb]
I’m not sure how that’s supposed to work though.
[kevinmarks] joined the channel
#
Loqi
[miklb] #76 Other Sites Saves all URLs on One Line
#
GWG
[miklb]: It's, once again, my mistake that needs to be fixed.
#
[miklb]
GWG, I had never used that so didn’t know if it was expected. I can take a look at the code, just wanted to confirm before digging in.
#
GWG
[miklb]: I know what needs to be fixed
#
GWG
The code was the same I used in Syndication Links
#
GWG
I fixed it there.
#
GWG
I didn't know anyone was using the Other Links box.
tantek and [chrisaldrich] joined the channel
#
[chrisaldrich]
In addition to the admin UI defaulting the output to a single line, it also appears that it's not outputting the <links> with rel="me" either.
#
GWG
I set it up because petermolnar, who wrote the original rel-me code in the Indieweb plugin, only defined Indieauth sites, and I thought people would want more than that.
#
[miklb]
GWG my point is, if you’ve already got a lot going on, just point me in the right direction and I can do a PR
#
[chrisaldrich]
GWG, I've used it in the past and know it worked properly.
#
GWG
[miklb]: It's in the class-hcard-user file.
#
GWG
I'm always in the middle of 50 things
#
GWG
I have a PR for Semantic Linkbacks, then some PR requests to go through, then back to Independent Publisher, mf2_s, and _s.
[jemostrom] joined the channel
#
[chrisaldrich]
I'm looking at my version running the same thing and I don't see the same problem phil is having AND the rel-me output is working properly on my site. Hmmmm....
#
GWG
[chrisaldrich]: You have any thoughts on Independent Publisher? I know you advocated its Indiewebification
#
GWG
It still should be saving as an array, not a string
#
[chrisaldrich]
I love Independent Publisher in a broad general sense.
#
[chrisaldrich]
I do wish it had a better nested menu set up (perhaps like 2015)
#
[chrisaldrich]
Another thing that would be nice in it would be to have a facepile of likes/favorites broken out for webmentions.
#
[chrisaldrich]
Though it does do a pretty good job of trimming down on the size and display of generic mentions compared to other indieweb friendly themes which show a lot of "cruft" instead of featuring "real" comments.
#
petermolnar
someone summoned me
#
[miklb]
chrisaldrich I disagree that an explicit field for micro.blog would be scope creep. I see the idea that the IW plugin is for ease of getting started. When the day comes that every site needs a rel=me link, that’s a bigger & better problem to solve IMO
#
[chrisaldrich]
I can certainly see your point, but the IW plugin is already adding a huge number of fields to the user page already... ease of getting started eventually isn't so easy if you're presented with 1000 fields to fill in....
#
[chrisaldrich]
And it's not very transparent based on that page what is being done with any of that data.
#
[chrisaldrich]
Given the prevalence of micro.blog users that are indiewebifying though, it may be worth it in the short term too.
#
[miklb]
“And it’s not very transparent based on that page what is being done with any of that data.” is a different problem
#
GWG
Well, some of the extra fields are for an h-card widget that never got built.
#
[chrisaldrich]
GWG, I thought there was a beginning of an h-card widget that got pulled out? Or was I dreaming?
#
[miklb]
I do not disagree that the profile field options cover a lot, but I would say that a useful now field for micro.blog is more important than “honorific title” ?
#
[miklb]
but consensus isn’t something common with the WordPress plugins these days
#
GWG
I'm confused. What field are we looking to add?
#
[miklb]
I suggested a field for your micro.blog URL
#
GWG
[chrisaldrich]: It is still in Syndication Links. I'd like it out, but I've never been able to add a nice one.
#
[chrisaldrich]
GWG, sorry, we brought a conversation from another slack channel back here...
#
GWG
[chrisaldrich]: Which room? I'll scroll through the logs
#
[miklb]
in addition to fixing the other sites field
#
[miklb]
microblog wordpress
#
GWG
[miklb]: Happy to pause and handle the error.
#
[miklb]
cool, because I suck at that fancy php
#
[chrisaldrich]
GWG, the data being saved to the "relme" field is being stored in wp_usermeta is stripping out the line returns for some reason
#
GWG
[chrisaldrich]: That is why I fixed it in Syndication Links
#
GWG
Oh, and [chrisaldrich], happy birthday
#
[chrisaldrich]
GWG: Thanks! I don't think it's even in my h-card...
#
GWG
[chrisaldrich]: I saw it elsewhre
#
GWG
But I could add that field. However, still not using it.
#
GWG
[miklb]: You've got review request.
#
[miklb]
looks good to me. There was a CI issue it looks like.
#
GWG
[miklb]: Unneeded spaces
#
[miklb]
what rules are run against that? WP coding standards?
#
[miklb]
needs to look closer at the release process for the IW plugins.
#
GWG
[miklb]: Yes
#
[miklb]
my phpcs is broken currently ?
#
[miklb]
also, see my comment about the Coding Standards repo
#
[miklb]
hmm. I guess pushing a tag isn’t enough.
#
GWG
I had that problem. It is supposed to be though.
#
[miklb]
I also made a formal “release” in GitHub but not sure what is supposed to trigger the deploy script.
#
GWG
I thought it was the tag
#
[miklb]
!tell pfefferle can you confirm the tag/release process for the indieweb plugin?
#
Loqi
Ok, I'll tell them that when I see them next
#
[miklb]
oh, it worked
#
[miklb]
pfefferle++
#
Loqi
pfefferle has 4 karma in this channel (31 overall)
#
Loqi
gwg has 8 karma in this channel (250 overall)
tantek, davidmead and [miklb] joined the channel
#
GWG
So, I submitted a PR to Independent Publisher, which leaves mf2_s to fix
[chrisaldrich] joined the channel
#
Loqi
gwg has 9 karma in this channel (251 overall)
#
GWG
[chrisaldrich]: Lots of small changes