2018-02-11 UTC
tantek, treora and [eddie] joined the channel
# 02:37 [eddie] Hmmm, I must be doing something wrong. I’m trying to post a JSON Syntax Micropub Request to Aperture and It’s breaking on XRay, saying undefined index: data.
renem, lovePhpoY and mblaney joined the channel
# 03:55 aaronpk I need to add some better error checking in XRay tho. It assumes you're giving it output from the mf2 parser right now
[eddie] joined the channel
# 04:51 [eddie] Ohhhh interesting. I must have blended h=entry and type: h-entry.
maingo and [eddie] joined the channel
barpthewire, [kevinmarks], [jeremycherfas] and jeremycherfas joined the channel
# 09:35 Zegnat What I did there is abstract a whole lot of logic I didn’t want to think about. So I depend on a standardised HTTP Client class and such.
# 09:38 Zegnat I created a KnownAPI class that does all the API work. The index.php loads the Composer autoloader to get access to dependencies (like the HttpClient).
# 09:38 Zegnat So all index.php does is initiate the KnownAPI class (which requires class dependencies and the user’s Known information) and then call the `create` method.
# 09:41 Zegnat Yes. That builds and sends the request. Should be able to convert that into curl code easy enough.
# 09:42 jeremycherfas OK, I need to go back to my first attempts and see what was happening. Thanks. I will bother you again later, I am sure.
# 09:42 Zegnat Most of the lines are just addHeader, so those go into a curl_setopt for CURLOPT_HTTPHEADER.
# 09:43 Zegnat Please do. Else you can always try to pull in the KnownAPI.php file. Just means you need to load a few dependencies through composer if you do.
# 09:43 jeremycherfas I had hard-coded my username and signature, so let me get back to playing with that.
KartikPrabhu joined the channel
# 10:09 jeremycherfas Looks like I too can post linked title and description now. But currently all hard coded.
# 10:11 Zegnat Awesome! Yeah. I was thinking yesterday that while the Micropub code messes some stuff up, it really should work with Known’s own API. So I gave it a shot :)
# 10:39 jeremycherfas The weird part is, I'm pretty sure that that is where I started off, and now I'm thinking that maybe I wasn't escaping things properly.
# 10:48 Zegnat Just happy to get you started with some working code again, no matter what went wrong previously :)
# 10:57 sknebel huh, Known doesn't show likes properly in their RSS feeds :/
# 10:59 sknebel wait no, it's somehow including broken bits for likes for normal posts
# 11:00 sknebel oh no, that's jeremys experiments with zegnats code ;)
# 11:01 sknebel got confused because my feedreader redirects me to the liked post, not the like-post
# 11:07 Zegnat Experiments with his own code, I don’t think he is running mine ;)
ben_thatmustbeme joined the channel
# 11:30 Loqi [aaronpk] There must be a better way to describe this then. The idea is that only the orderings of the provided items would be affected by the operation. Some examples:
given: `[a b c d e]`
command: `[c b]` ("move `c` up")
result: `[a c b d e]`
given: ...
# 11:51 Zegnat Oh, yeah, that makes a lot more sense as a system
# 11:51 Zegnat I was just wondering how the thing aaronpk wrote could be applied at all. Mostly struggling to understand the more-than-two case.
raretrack joined the channel; mblaney left the channel
dgy joined the channel
# 15:30 aaronpk Zegnat: I implemented that yesterday in Aperture, it wasn't too hard.
# 15:30 Zegnat I should check out the code. Maybe that one example you gave with three items just threw me off. I wasn’t sure how to get from the input to the output with that command
# 15:43 aaronpk I suspect the vast majority of the usage will be swapping two adjacent items or specifying the new order of the full list, but it's convenient that there is only one method needed for both
# 16:15 Zegnat I am not sure I understand the code you put into Aperture, aaronpk. Isn’t that only accepting a full array of the new order?
snarfed joined the channel
# 16:18 aaronpk since there is a database column storing an integer used for sorting, that function swaps the integer with any of the other channels provided in the input
# 16:19 aaronpk yes, it takes an array of the new order of the channel IDs
# 16:19 Zegnat I was most interested in the function that enables f([a b c d e], [e d b]) = [a e c d b]
# 16:20 aaronpk line 41 gets the current order, the first array in your example f
# 16:20 aaronpk actually no I guess it's not an analogous version of your f
# 16:21 aaronpk this only ever touches the channels that are being modified
# 16:21 GWG Wondering if using two time and date selectors was the best for duration
# 16:21 Zegnat f(arrayPerOldSort, reorderCommand) = newlySortedArray ; basically taken from your comment in the GitHub issue
# 16:22 Zegnat And I am having a hard time seeing how f() works there. But maybe your example was a bit far-fetched?
# 16:22 aaronpk the function I wrote will work with the `e d b` example, but that's because it's not strictly doing `f([a b c d e], [e d b]) = [a e c d b]`
# 16:23 aaronpk it's pulling things from the database and writing back to the database, rather than accepting input and returning the new sorted output
# 16:24 Zegnat Yeah, I’m starting to see that now. Finding it hard to visualise that though. Maybe I should setup a Aperture db instance just to get the clear picture.
Zegnat, barpthewire, tantek and snarfed joined the channel
# 17:14 Loqi Ok, I'll tell them that when I see them next
[eddie] joined the channel
# 17:16 tantek hah, my title element on my pay page has the same "mistake" (omitting amount)
# 17:16 aaronpk I might need to move the logic in my pay pages into the backend of my site instead of having it all be JS code
# 17:17 aaronpk I did it all in JS at first to avoid having to make a bunch of special-case custom backend code
# 17:17 aaronpk but it's been pretty stable now, so might be worth solidifying
# 17:19 aaronpk if I move it to the backend then I can do things like turn off the twitter card or make it better, and change the title tag to include the amount
# 17:30 tantek now that I created the Github Bridgy org, I noticed something else new in the oauth prompt for Bridgy Pub to Github
# 17:31 tantek the Bridgy org is listed there, along with a [ Grant ] button to its right
# 17:31 tantek in addition to the org above it which has a [ Request ] button
# 17:31 tantek the other orgs listed before those have no such buttons
# 17:33 tantek I just got it yesterday (released from a squatted account)
# 17:33 sknebel then it makes sense you have the right to give apps access to it I guess
# 17:35 sknebel others would have to do "request" and then you could allow them to do so as well I guess
KartikPrabhu joined the channel
# 17:41 [eddie] Yeah the request is because you don’t have the authority to grant access, so the request button will send that request to the org admins
# 17:42 [eddie] Grant appears when you have authority to grant access
KartikPrabhu joined the channel
# 18:04 [eddie] aaronpk, about Microsub spec: Yeah, I think so. I think notifications remain as they are. But I think Home is just another channel in the Microsub Server. No special client treatment.
# 18:05 [eddie] But a Microsub Server is required to start with a single Home channel (uid is whatever the Microsub Server wants)
# 18:05 aaronpk that also means omitting the notifications channel from the list makes more sense
# 18:05 aaronpk although one problem with that is how do I return the number of unread items in that channel
# 18:06 Zegnat reflexively went to microsub.spec.indieweb.org but that isn’t a thing
tantek joined the channel
# 18:07 [eddie] Hmmm I would be okay with notifications being returned in the list of channels for information wise. I think the main issue is sorting. It seems strange to have notifications in the sorting mix
[kevinmarks] joined the channel
# 18:09 [eddie] Would it be weird to always return notifications as the first channel, and to define that a Microsub server ignore any attempt at placing the uid notifications in the re-order call?
# 18:09 [eddie] So if I sent [a, b, notifications, c, d] the server would just remove notifications from the list
# 18:10 [eddie] And have the spec define “special channels such as notifications can’t be ordered”
# 18:10 aaronpk one benefit of that is if clients *dont* include any special handling for the notifications channel it will still show up
# 18:10 [eddie] That is true. And at the top which is a good place to have notifications
# 18:14 GWG Anyone available for a moment to talk Link Preview data?
# 18:14 GWG I'm still trying to make sense of something
# 18:17 GWG I'm taking a URL and parsing it, and using that to present a preview of same.
# 18:20 GWG So, I'm getting a lot more than just name, author, and featured right now. I'm capturing all the elements used in comment presentation as well
# 18:20 GWG summary, published date, updated date, categories...
# 18:22 GWG So, I've been storing featured image, published/last updated date, and categories/tags for some time, but haven't actually displayed them. I am brainstorming displaying them
# 18:22 GWG But, is there such a thing as too much data in a link preview?
# 18:23 GWG Also, I set it all up to work as a citation.
# 18:23 GWG So, in different contexts, site name, publication, and album(for posting music) are all the same data point.
# 18:23 GWG I'm wondering if I made it too complicated
# 18:25 [eddie] I think there is such a thing too much preview data. Not for storage as much as display. I think the key is not overloading the display. But I think the data is useful because it can inform the display
# 18:25 GWG Well, what I did in weather is I hid a lot of the fields from view, but still stored them.
# 18:25 [eddie] I think that is useful to store them but hide them
# 18:26 GWG Fields as in, in the editor, not the actual post, because I'm not currently displaying some of them, but held them because I want to someday
# 18:26 GWG I'm rethinking every single one of them
# 18:26 [eddie] Does it allow someone to edit it? Or just see what they are commenting on?
# 18:27 GWG I actually allow it to be viewed/edited in the post UI.
# 18:27 GWG Mostly edited. Not everything parses as expected
tantek joined the channel
# 18:27 GWG But I want to hide that more to encourage people to not worry about it so much.
# 18:28 [eddie] Yeah I think that’s good. I don’t know much about Wordpress UI but it might even be nice to hide it behind a “edit preview data” button where it takes an extra step to edit it. Thus discouraging the editing unless it’s required
# 18:28 GWG I also included Start and End Date/Time, which is currently only used to calculate duration, because I wanted to have it for events later on...but I'm thinking I might have been better off with a duration picker.
# 18:29 GWG [eddie]: I'm asking in here, over the WordPress room because it isn't a WordPress specific discussion
# 18:30 [eddie] That’s tough, for events I think start and end time IS useful. But I can see how having it there for every most might be a bit much
# 18:32 GWG So, I was thinking of adding a duration picker and showing/hiding the start and end date as needed.
# 18:32 GWG I do specific post typing, not implied
# 18:32 GWG So, there is a selector I can use
# 18:32 GWG More JS than I normally do, but the complaint j12t made was that no one could figure out what properties should be set for what type
# 18:33 [eddie] Yeah I think that’s a huge thing to think about. Definitely a good choice to start hiding things based on type
# 18:33 [eddie] The less cognitive work when a person posts the smoother the flow
# 18:35 GWG All I ever want to do with JS is things like show/hide/AJAX. I really don't like sites where things don't work without JS.
# 18:36 GWG The way I'm changing it, everything will show without JS
# 18:37 [eddie] Basically make everything show and then hide it using JavaScript on initial load
# 18:37 [eddie] So you mentioned replacing start/end with duration
# 18:37 Loqi It looks like we don't have a page for "duration for on other post types" yet. Would you like to create it? (Or just say "duration for on other post types is ____", a sentence describing the term)
# 18:38 GWG [eddie]: Duration is used for music/video/reading, for example.
# 18:38 GWG So, I'll likely set it that if start/end is set, but duration isn't, to derive one from the other still.
# 18:39 sknebel sounds good. If you have the space, quick buttons for hours and minutes (:15, :30, :45, :00) also can be quite effective
# 18:40 aaronpk [eddie]: one change with removing `default` as a known uid is now the channel parameter is required for every command
# 18:41 dgold is there supposed to a notifications channel in a msub server?
# 18:41 aaronpk in Aperture? I thought I made it create that when a new user is created
# 18:41 [eddie] aaronpk: ohhhhh hmmm that does seem an unfortunate side effect 😕
# 18:42 dgold aaronpk: I have 'home' and one that I added myself
# 18:42 dgold home doesn't appear to have anything in it
# 18:42 aaronpk dgold: weird, i'm gonna have to look at that. it was supposed to create two channels for you when you first logged in
# 18:42 [eddie] dgold: Do you see home in Aperture or Indigenous?
# 18:42 [eddie] Just because I add the Home in there in indigenous automatically
# 18:43 [eddie] Yeah so your aperture doesn’t have either Home or Notifications
# 18:43 dgold in Aperture, when I created my first user, there was nothing there apart from aaron's apologia
# 18:44 aaronpk there's an event that is supposed to run after a user is created, but that would only work if it's created from code
# 18:45 aaronpk so I should make a command to create the first user, since I disabled creating users by logging in via indieauth
# 18:46 aaronpk [eddie]: how annoying will it be to always have to specify a channel in requests?
# 18:51 aaronpk having the notifications channel in the list also means you don't have to localize the name on the client
# 18:51 aaronpk oh you can remove Pusher from that, I realized I had just left in the default .env config from Laravel. I don't actually use Pusher for anything
# 18:53 aaronpk whoa do your article pages also show other posts?
# 18:54 dgold ah - I linked to that before the publication thing ended its work
# 18:55 dgold if you follow that link again, it should just show the single post
# 18:55 aaronpk I still see a bunch of stuff after the last line "visit your new site and sign in"
# 18:56 Loqi [Daniel Goldsmith] WebSub Part II: Aperture
# 18:57 dgold yeah, if loqi sees that, then its your local cache
# 18:58 dgold ok, for some reason, the fist is redirecting to my homepage
# 18:58 sknebel that's odd, I thought it didn't a few moments ago
# 19:06 Loqi [Daniel Goldsmith] WebSub Part II: Aperture
# 19:11 aaronpk mentioning github issues in the wiki changelog means the wiki edits show up on the github thread
# 19:14 tantek aaronpk but only the first of the two Microsub-spec edits above
# 19:15 tantek also if that's the case, how come the IRC chat log mirrors on github are not also showing up in the issue?
# 19:16 aaronpk I made the wiki sync script use the wiki changelog as the git commit message
# 19:16 aaronpk tho it only commits every 5 minutes, not on every wiki edit, which is probably why that second one didn't show up
# 19:16 aaronpk wiki changes get clustered in 5 minute chunks to git. i'd like to have each wiki edit be its own git commit but that is a lot harder
# 19:17 tantek 5 min is likely enough granularity to catch most edits by themselves
[jeremycherfas] and [kevinmarks] joined the channel
KartikPrabhu joined the channel
# 19:37 aaronpk aw man, i'm already doing the thing I didn't want to deal with by having other users of thi
# 19:38 aaronpk I have to make a migration that will fix things up for you all rather than just hacking my own account up
# 19:38 tantek oh? adding features that you don't use yourself?
# 19:44 Zegnat To be fair, aaronpk, using migrations (if you are talking DB) with your use of Laravel is also just a good practice
# 19:44 aaronpk I would have just moved my notifications channel to the top of the list on my account. but instead I made a migration to adjust that for all the user accounts.
# 19:47 aaronpk not a huge deal, it's just that this is exactly the thing I was trying to avoid at this early stage of the project, to make it so I can move quicker
# 19:54 Zegnat To be fair, all “users” know that this is all unsupported.
# 19:54 Zegnat (To be fair is turning into a new starting sentence for me, I need to stop that.)
# 19:57 aaronpk yep but it'll save me time in the end to make this migration instead of having to give instructions to people in IRC later :P
# 19:57 Zegnat I’ll file an issue against your kindness, is there an aaronpk/aaronpk repo?
# 20:01 dgold aaronpk: I know that Aperture is early alpha - I have no expectations.
# 20:13 Loqi kartikprabhu has 9 karma in this channel (161 overall)
[eddie] joined the channel
# 20:34 [eddie] aaronpk: ditto to what is being expressed. Happy to alter my database how I need to
# 20:44 dgold [eddie]: are you using indig as a m.blog reader?
[kevinmarks] joined the channel
# 21:21 [eddie] dgold: I want to build native m.b support. But for now, I used Aperture to subscribe to my m.b JSON timeline feed
# 21:21 [eddie] So one of my Aperture channels has all the posts from m.b
AngeloGladding and snarfed joined the channel