#sgregerHi all! I am terribly stuck, trying to track down a bug in my Kirby Micropub plugin. When sending tags using Indigenous, they show up in the parsed request body, but with Quill they don't. Any pointers to what is the difference in how these two send their requests?
#sgregerI am referring to the category[] values in the encoded request ...I just can't figure out why they don't show up in the request body coming in from Quill, but they do when I use Indigenous.
moppy and jonnybarnes joined the channel
#sgregerActually, after more digging, I just noticed that the tags are not included in the multipart request the Quill UI sends to quill.p3k.io - the "Request payload" only contains fields "content", "mp-slug" and "null". Tested on FF and Chromium, both times the form data sent by the UI contains this mysterious "null" field but no "tags". Is this potentially a bug in Quill?
jonnybarnes, nsh, smacko, jeremycherfas, dhanesh95 and jamietanna joined the channel
#jamietanna[m]sgreger I've just had a look my side, and I see `&category%5B%5D=foo&category%5B%5D=bar` being provided in the request, so it looks like Quill is working, but you may be expecting the wrong field? In Micropub we use `category` or `category[]` for tags
[jgmac1106], jonnybarnes and sgreger joined the channel
#sgregerjamietanna[m]: Thanks for confirming Quill + tags work for you! My problem must be something else, then. I just tried with FF80/Linux, FF80/Win, and Chromium85/Linux:
#sgreger...two different machines, three browsers, always the same issue (so I dare ruling out an adblocker or the like breaking some JS in the frontend). But if it works from your machine, I'm out of ideas what might be the cause.
#jamietanna[m]Ah interesting, I think you need to hit tab when you've added a tag, rather than comma separate them sgreger
#sgregerHmm, that moves the cursor to the next input field (slug) :s
#ZegnatHi sgreger! Long time no seen :) Hope you are well, apart from the micropub struggles. Lets see if Sink is still up and running, might be able to see what Quill posts there
#ZegnatHmm, as soon as I write a comma in Quill, the tag becomes a little grey tag with an x mark
#ZegnatNo idea where that null=null is coming from, that does seem like a bug. But yes, submitting the form I previously screenshotted: https://i.imgur.com/cfPE6UV.png
#sgregerzegnat: that's what I expect and remember but for some weird reason it does not happen. But with all of you confirming it works, it's quite obvious something on my end!
#ZegnatWhat happens if you log Quill into https://sink.zegnat.net on your end, sgreger? Just to doubly-doubly check that it is not the Kirby endpoint making things weird.
#sgregerZegnat: it's a miracle - when I log in with Sink it works!
#sgregerSo there's definitely something fishy with my plugin (that's good news; it means I can fix it).
#ZegnatHuh. Quill really hsould not make a distinction there xD
#ZegnatI haven't touched the Sink codebase in years, but pretty sure it does nothing special. I think it does not even provide the config endpoint for a client to check support
#sgregerNow this is getting even more bizarre. But at least all of above interactions prove that Quill itself is not the problem.
#sknebelhmmm... have you implemented anyhthing regarding the query for already existing categories?
#sgregerConfig endpoint would have been my next guess. Maybe Quill expects some signal from Kirby that categories are supported? (Now I need to dig into code from April to see what my endpoint actually communicates)
#sknebelno idea if quill supports that, but that could cause confusion somewhere
#sgregersknebel: I'll look into that, thanks! ...also: Hello - long time no see :)
#ZegnatAs far as I remember, Sink flatly does not support config. So that could be a difference between it and the Kirby endpoint. Let us know if there is anything else we can do to help
#ZegnatReally missing travel over here, hope we can get together at a German event again in some future timeline, sgreger
#sgregerMy endpoint indeed does reply to q=config requests, so maybe there's some error in that data that throws off Quill (and causes the tag tokenization to fail). I'll look into that later - thank you so much for leading me onto the (hopefully) right track!
#aaronpksgreger: Can you check the dev console and see if there are any errors reported? If there's a JS error then it might prevent the tokenfield thing from running
jonnybarnes and sgreger joined the channel
#sgregeraaronpk: no console errors, no missing files (404s etc.) in the Network tab, nothing. Just no tokenfield. I'll try some things with the config query and report back (as it works with all non-Kirby sites).
#sgregerindeed. it just fails silently. but when i use zegnat's sink tool, the UI is fine, so it must be something about my endpoint
#sgregergetting there: if i remove the `post-types` array from q=config, it works. my guess is that, since there was no post type of `"type": "note"` present in that array, Quill shows the Note editor window while believing notes are not supported?
#sgregerso, with no `post-types` array present, it works fine and the tags make it all the way into my Kirby post. interestingly, once I re-add "post-types": [{ "type": "note", "name": "Default" }] the tokenfield doesn't work again.
#ZegnatThere should be a page within Quill that shows you the supported types. Maybe when you click your name or something? Can you see if that page seems to work?
#sgregerWell, there is the Settings page that correctly indicates "supported post types: * Note (note)" and Quill also correctly only provides the link to the Note editor window in the footer
#sgregerMy q=config response is now as barebone as can be: just the `media-endpoint` URL string, an empty `syndicate-to` array and the `post-types` array with the Note entry.
jeremych_, swentel and dhanesh95 joined the channel
#sgregersknebel: well, it partially was. returning an invalid value in the config array (post type "default") rightfully caused some hiccups. the problem when returning the correct "note" post-type might be something in Quill.
#sknebeltrue, but even the former is IMHO a client bug if it results in an input field being silently ignored
#sknebel(especially since "category" is not associated with any post type)
[tonz], [grantcodes] and jonnybarnes joined the channel
#sgregerthanks again to all of you for the assistance!
nickodd, [LewisCowles], [tw2113], geoffo, [schmarty], jessehattabaugh_, lahacker, curiousaboutindi, jonnybarnes and leg joined the channel; nickodd left the channel