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.
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.
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
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.
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
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)
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
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
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).
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.
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