#wordpress 2021-02-03

2021-02-03 UTC
[KevinMarks], [schmarty], beko, [chrisaldrich], [benatwork], [Emma_Humphries] and [pfefferle] joined the channel
#
[pfefferle]
I had some issues with post-kinds sending webmentions, is this a common issue? or am I doing something wrong? https://github.com/dshanske/indieweb-post-kinds/issues/326
#
Loqi
[pfefferle] #326 webmention_links filter does not seem to work
#
[pfefferle]
I am working on a presentation and I wanted to show the post-kinds plugin, thatโ€™s why I am a bit pushy ๐Ÿ˜ž
#
[pfefferle]
should prepare my slides earlier ๐Ÿ˜ž
[indieweb653] joined the channel
#
[indieweb653]
[pfefferle] whatโ€™s the kind of the specific post?
#
[pfefferle]
i tested it with reply and like
#
[indieweb653]
I tested locally and there seems to be a problem with nested properties, try adding after the line `$cite = $kind_post->get_cite();` this piece: `$cite = $cite['properties'];`
#
[pfefferle]
sadly not
#
[pfefferle]
it seems that the get_cite is empty
#
[pfefferle]
ok cite is not empty any more
#
[pfefferle]
but you are right
#
[pfefferle]
there are a lot of nested properties
#
[pfefferle]
[indieweb653] it worked!
#
[pfefferle]
edited the wrong line!
#
[pfefferle]
you saved my day! thanks a lot!
#
[pfefferle]
[indieweb653]++
#
Loqi
[indieweb653] has 3 karma over the last year
#
[pfefferle]
made a second merge request!
#
GWG
[pfefferle]: Thanks for finding that webmention bug, I see where I introduced it
paul3 joined the channel
#
GWG
[pfefferle]: Implemented different fix. Try it now?
#
GWG
When I changed from the older mf2_post class to the kind_post class, the signature output of get_cite changed.
[pfefferle] joined the channel
[KevinMarks] joined the channel
#
GWG
[pfefferle]: Standardizing storage as mf2 json is what caused it. You pointed out I was inconsistent on doing that during the mf2-feed plugin discussions
#
GWG
Either way, pushed a new version to everyone.
#
[pfefferle]
will test it asap
#
[pfefferle]
thanks [dshanske]
#
[pfefferle]
seems to work fine! thanks [dshanske]
#
GWG
[pfefferle]: Happy to fix a bug
[schmarty] joined the channel
#
[pfefferle]
and I felt a bit productive today ๐Ÿ˜‰
beehiveth joined the channel
#
beko
oh. oh oh. post-kinds update that fixes a bug sending Webmentions. That sounds totally like my issue I never found out.
#
sknebel
how does WP-land do with automated testing?
#
sknebel
(actually, I wonder if a mostly site-tech agnostic thing of "point at a site with Micropub and WMs, make a bunch of automated posts, verify WMs got sent, parse site to verify content survived" would be possible and useful)
#
beko
think that's a Docker rabbit hole at the moment.
#
sknebel
the full-site thing yeah, pretty much. hopefully doing tests of components inside WP is easier(?)
#
beko
Heh. Don't think so. I know that GWG was going to check this out. It does help a lot when loading the php codesniffer rules for WordPress though. This checks also for some common mistakes resulting in more robust code.
#
beko
oh wow. that really fixed my outgoing Webmentions. Hoooray. No more manually forging curl requests!11eleven Thanks GWG and [pfefferle]
#
[pfefferle]
๐Ÿ˜‰
#
GWG
I am testing challenged
#
GWG
But I do some tests
#
GWG
Clearly I need to do more
#
GWG
It isn't so much the writing as figuring out what to test
#
GWG
I test many low level functions
#
GWG
So, setting and getting from complicated data structures I try to test
#
beko
In another project we used Wallaby.js that integrated with the editor and displayed every line for test coverage and result. That was nifty.
#
GWG
I edit in vim
#
beko
brave
[Murray] and [tantek] joined the channel
#
GWG
I'm old school
#
GWG
I started coding HTML in 1995 in pico
#
GWG
I do need to write some unit tests for my latest feature. Location taxonomies...as it is a different behavior than default
#
[pfefferle]
btw. simple location and post-types throws a lot of errors/warnings when fresh installed
#
[pfefferle]
there seems to be something wrong with default settings
#
[pfefferle]
it disappears when you hit save in the settings areaโ€ฆ at lest for post types
#
[pfefferle]
and yarns seems to break the complete site if micropub and/or indieauth is not installed?!?
#
[pfefferle]
I am in debugging mode today ๐Ÿ˜„
#
GWG
[pfefferle]: Re Yarns...jackjamieson has been busy
#
[pfefferle]
๐Ÿ˜ž
#
[pfefferle]
but maybe microsub is also something for the friends plugin ([indieweb653] ) ๐Ÿ˜‰
#
GWG
Re default... maybe I should purge my options and investigate why the default registration of options isn't working
#
GWG
[pfefferle]: He finished his doctorate and then moved to Japan. He said he hoped to resume once settled.
#
GWG
[pfefferle]: Any thoughts on Simple Location bugs? I'm finishing a bunch of bug fixes for it and a new feature
#
[pfefferle]
I can re-check on a clean system with wp_debug enabled later today
#
[pfefferle]
but seems to be also a default settings issue
#
[pfefferle]
I disabled it on my test server for the time beeing
#
GWG
It's the most niche plugin.
#
GWG
Only a small subset of people care about adding weather, for example, to their posts.
#
[pfefferle]
yes, had no need so far to do so ๐Ÿ˜‰
#
GWG
[pfefferle]: The map part is useful
#
GWG
If you like posting photos, which I do
#
GWG
It also fixes exif timestamp errors Core won't
#
GWG
Won't as in, ticket not tackled
#
[pfefferle]
I am not that personal on my siteโ€ฆ
#
GWG
I know
#
[pfefferle]
๐Ÿ™‚
#
[pfefferle]
I try to keep personal stuff private
#
GWG
I'm waiting for one person who tells me how much they love being able to backdate a post to 1977 and get the local weather conditions...
#
[pfefferle]
notiz.blog is only my semi-professional subset ๐Ÿ˜‰
#
GWG
[pfefferle]: Me too..just a different extent
#
GWG
I rarely post photos of people, just places
#
[pfefferle]
ok, I can think about doing that
#
GWG
[pfefferle]: I felt awkward when I pulled out a photo of [tantek], [schmarty], and [manton] from IWC Austin and added it. But they all consented as part of the IWC photo policy.
#
GWG
Still, rarely post photos of people
#
GWG
I like doing things like this...
#
GWG
Auto generated map of the days post, from June 28, 2009
#
[pfefferle]
ok, I also have no problems to post these kind of thingsโ€ฆ they are part of my semi-professional self :)
#
Loqi
Gestern hatten wie unseren 10. Homebrew Website Club in Karlsruhe ๐ŸŽ‰๐ŸŽŠ๐Ÿพ OktoberSeptemberAugustJuliMaiAprilMรคrzFebruarJanuar(Ja, wir haben im Juni vergessen ein Bild zu machen!) Danke an den harten Kern, Daniel Ehniss, Jens Strobel, Dani...
#
[pfefferle]
[dshanske] nice!
#
GWG
I haven't done person tagging yet to my satisfaction
#
GWG
[pfefferle]: Soon, stay tuned for /location/DE/Karlsruhe
#
GWG
Actually, forgot region
#
GWG
Not sure what region it would show up in
#
GWG
Geocoders are inconsistent
[arush] joined the channel
#
[arush]
Hi all.
[benatwork] joined the channel
#
GWG
Hi, [arush]. What's new?
#
[arush]
Re: Simple Location bugs: I'm unable to add location to a post when adding it from the native WP add-new-post screen, but it works fine with Micropub. Not getting any visible errors though. Re: backdating posts and adding historical weather: I haven't played with this yet mostly because I haven't figured out how to do it, but that would be fun. I have a lot of backdated stuff on the site and love adding weather/location to my posts.
#
[arush]
When I try to add weather/location to posts natively, I can add the data but then unable to publish. Press the button, nothing happens.
#
[arush]
Publish/update.
#
[pfefferle]
ah, that was the error! I had something like โ€œno geocodeโ€ warning when I tried to create a new post
#
[arush]
Yeah something like that I think, but not sure because host's access and error logs display crashes my screen reader every time. Still trying to get a VPS set up to host indieweb apps plus this WP install to just take WPEngine out of the equasion.
#
[arush]
They're great for stuff that must stay up and running, like client work. Not so much for experimental stuff. ๐Ÿ™‚
#
[arush]
Oh, and Gutenberg work with post kinds is on hold right now until the inability to edit blocks with AT thing is fixed. Currently arguing about that one with the devs so not sure when this will happen.
#
[arush]
And working on getting new computer set up to do more dev. Hasn't gone as smoothly as it should but I think things are finally settling down and getting straight.
#
GWG
[arush]: I assume you are using the imperial measurements? If so, I found the bug
#
GWG
I have it set to fix in the next release
#
[arush]
@gwg yes using imperial.
#
GWG
When I changed the system over the convert to fahrenheit, I didn't round the conversion and the form is limited to 2 decimal points
#
GWG
So if you edit the measurements to round it will work
#
GWG
[pfefferle]: To do reverse geocoding you need a geocoding provider
#
GWG
You can use the airport geolocation provider which requires you to add an airport code to the address field to get a location
#
[arush]
@gwg also, what are the specifics you want for people to contribute to plugins, other than the obvious needs-to-happen-on-Github. I have some accessibility fixes for some of the screens for post kinds and simple location, but don't want to mess up work flows or anything. So, do we need to use anything like composer ETC. or is there a readme file for contributors?
#
[pfefferle]
[dshanske] it was a php error/warning
#
GWG
[arush]: I only use composer and Grunt to bring in dev dependencies.
#
GWG
I picked it up from [pfefferle]
#
GWG
Considering that all the plugins I wrote us a similar style to his because I picked it up from him, we probably should write a common contributors file
#
GWG
Saves duplication
#
GWG
[arush]: For now...I will consider anything... what do you have in mind first?
#
GWG
I test all PRs
[schmarty] joined the channel
#
GWG
I always prioritize fixing things that I agree are broken
[snarfed], [Ana_Rodrigues] and [Rose] joined the channel
#
[arush]
@gwg On the settings screens, there are some checkboxes with labels that don't seem to match. For example, on the Simple Location settings screen, under general, THERE'S SOME TEXT THAT SAYS "When Making a New Post, update the Author with Posts Location" BUT IT DOESN'T HAVE AN ASSOCIATED CHECKBOX. The checkbox has a label of "Show Maps on Home and Archive Pages Not Just Single Posts". I think that's supposed to be the second checkbox, but
#
[arush]
the second one has a label of "Automatically lookup location from supported geolocation provider for Micropub posts" and the third checkbox has a label of "Unit of measure".
#
GWG
[arush]: That might be a mark-up problem
#
GWG
The box is there. I'll have to look at the coding
#
GWG
I probably copied the same checkbox code to add new boxes
#
[arush]
On post screens, Simple Location has some spinboxes, but it's not clear what values you're editing or how they should be edited. Elevation, for example. It moves by 1, 2, 3, ETC., but obviously that's supposed to reflect unit of measure and so should be moving by some number of feet/meters. I'm assuming some interval, but not sure. Post Kinds kind metabox has some extra ARIA, so the label reads twice. "read read", for example, along with
#
[arush]
text below each box reading the same thing.
#
[arush]
So anyway I figured since I use these plugins like mad, I'd fix that kind of stuff and try to leave enough info in comments/commit messages so that if those things are present across the indieweb suite, the fix could be easily copied.
#
GWG
[arush]: The form field bugs with numbers makes me think that I should classify them as text. I think I show units of measurement in parentheses but probably should add to the labels
[calumryan] joined the channel
#
GWG
Need to add to that string I believe
#
[arush]
@GWG Also, I wanted to ask about the photo post kind, specifically the properties section. If you want to parse a URL, which one is it expecting? File? Attachment? How do you get it to add the location of a photo to the appropriate box, or are we supposed to enter that manually? I'm thinking not file because if you insert a URL to the jpeg you get a message back saying something to the effect that it's not valid/not supported.
#
[arush]
I suppose a shorter version of that is: What's your workflow for posting photos?
bishop_bautista joined the channel
#
GWG
[arush]: I use the button that pops up the Media Library chooser
#
GWG
But it is the attachment URL. Although I should change it in future
#
GWG
I probably should fix the not supported issue too
#
[arush]
@gwg OK, so in other words the photo is supposed to go in the actual content area, and it's data should be taken from the attachment URL for the properties section.
#
GWG
[arush]: Actually, it's complicated
#
GWG
If you put a photo into the content area, it didables usage of the box at the top
#
GWG
It's a compatibility check
#
GWG
Otherwise it will use the box at the top, but the way to add is using the add media button, which means I probably should hide the URL field
#
GWG
It's definitely a poor interface
#
GWG
I keep meaning to change it, but then I think Gutenberg...
#
[arush]
Oh. I didn't see that happen, but then I was updating the post to get it to work and finally landed on add photo to the content area because it needs alt text. Hang on let me try with another new photo.
#
GWG
[arush]: I'm glad of your interest in improving it
#
GWG
Sometimes I think I'm too close to the problem
#
[arush]
Oh. There's a second attach media button I never saw until now. Doesn't look like it's in the tab order, which would be why I missed it because I always tab through that section.
#
GWG
[arush]: One is the one for the post kind, the other is the content one WordPress always adds
#
GWG
The post kind one only appears on photo, audio, or video.
#
[arush]
For some reason that added two photos, but I think that's a bug in the media dialog when you're running classic editor. If you exit forms mode at the right spot in the process, the dialog disappears and you have to start over again, but you can't see your previous work. Probably won't get fixed because Gutenberg is taking up all the oxygen.
#
[arush]
Kind of want to play with iOS shortcuts for photos, but don't want to do that since pname is micropub for alt so will have to wait till that can be put back.
#
[arush]
name
#
GWG
[arush]: It shouldn't be..I think there is an alt-text Micropub format in the spec
#
[arush]
Damn. Can't type. P-name.
#
[arush]
I'll have to review that. When I was doing some brief googling to try to find something, all I found was p-name for alt. But that could be old info.
#
GWG
To include alt text along with the image being uploaded, you can use the Microformats 2 syntax to specify both the image URL and alt text for the photo property. Instead of the value being simply a URL, the value is instead an object with two properties: value being the URL and alt being the text.
#
GWG
So, only works with json syntax
#
GWG
It looks like Micropub for WordPress supports it
lahacker joined the channel
#
[arush]
Thanks for passing this along. Sorry for the delay in response. Jaws took a dive.
#
GWG
[arush]: Quill has an alt text field I think
jamietanna, [KevinMarks], [benatwork], [tantek], gRegorLove and [arush] joined the channel
#
[arush]
@gwg yeah I think it does.
[snarfed] joined the channel