| [20:08:45] | <edbob[m]> | managed to get webhook to do what i needed, for simple test case - much more to be done. but the "drupal transactions" issue comes into play here apparently. my webhook logic fires and i fetch the record, but i get "old... (full message at <https://matrix.org/oftc/media/v1/media/download/AfvjyruKoVhe-MieBkwtZq5r...) |
| [20:13:36] | <mstenta[m]> | edbob: wow! |
| [20:14:25] | <mstenta[m]> | Glad to hear that module is working... sort of... 😅 |
| [20:16:05] | <edbob[m]> | yeah i'm not sure it will be worth "recommending" yet. really it's overkill since i just need to know entity_type--bundle and the uuid, plus event type (create/update vs. delete). and it sends the full record basically.. which it's just occurring to me, i should inspect that instead of re-fetching the record from API... |
| [20:17:04] | <edbob[m]> | that just sucks a little b/c my sync already has logic to parse a record from the API, but their webhook payload has a different format entirely |
| [20:17:44] | <mstenta[m]> | Ahh yea |
| [20:18:09] | <edbob[m]> | time for another break then.. :) |
| [20:18:54] | <mstenta[m]> | Maybe we can try out the patch from that issue on a dev call |
| [20:19:39] | <mstenta[m]> | It looks big though... need to look closer later... |
| [08:02:32] | <farmBOT> | @farmOS@fosstodon.org => The #farmOS monthly community call is today at 2pm Eastern. All are welcome! https://farmos.org/community/monthly-call/ => https://fosstodon.org/@farmOS/116209867881892443 |
| [08:02:54] | <mstenta[m]> | <edbob[m]> "that just sucks a little b/c..." <- edbob: good morning! curious about this... do you mean that it is not in the same format as the JSON:API endpoints? |
| [08:03:02] | <mstenta[m]> | farmBOT: hi! |
| [08:03:55] | <mstenta[m]> | edbob: Just saw this in that module's issue queue - maybe related? https://www.drupal.org/project/webhooks/issues/3168470 |
| [08:06:03] | <mstenta[m]> | That module looks more complicated than we would probably need, in general... it has the ability to receive webhooks too, and a bunch of admin config UI etc... probably super flexible! Probably more than we need... :-) |
| [08:06:14] | <mstenta[m]> | I bet the actual code for dispatching webhooks is simple. |
| [08:06:32] | <mstenta[m]> | Maybe we should sketch up a simple farm_webhook contrib module to experiment with. |
| [08:08:15] | <mstenta[m]> | If that's something you'd be interested in working together on, I'd be happy to get some of the basic code for it started so you could hack on it |
| [08:10:10] | <mstenta[m]> | Basic requirements, in my mind, would be:... (full message at <https://matrix.org/oftc/media/v1/media/download/AbcWrxS1x8K7cG0Fe5nUJ7v5...) |
| [08:12:02] | <mstenta[m]> | Of course, requirements will probably grow from there... ;-)... (full message at <https://matrix.org/oftc/media/v1/media/download/AeGcVuWfQw1xmsZaz-SBXT-o...) |
| [08:14:27] | <mstenta[m]> | Maybe that module already handles those things... 🤷😅 |
| [08:21:05] | <mstenta[m]> | <mstenta[m]> "It looks big though... need to..." <- I left a comment on https://www.drupal.org/project/webhooks/issues/3572436 - seems like a lot that patch's complexity is because they are making it configurable. Seems like that's unnecessary, and it could be a lot simpler, but we'll see... maybe there's a good reason 🤷 |
| [08:24:11] | <mstenta[m]> | <mstenta[m]> "Someone make this test failure..." <- pcambra: I'm stumped on this. Do you think it could be an issue with Drupal.org GitLab testing infrastructure? The tests pass in my local testing environment. |
| [08:24:53] | <mstenta[m]> | I would ask in Drupal Slack, but Slack doesn't load in Firefox/Chromium for me anymore 😮💨 |
| [08:25:59] | <mstenta[m]> | (They never should have moved off of IRC IMO 🙃) |
| [09:12:25] | <symbioquine[m]> | Could it be a DST "bug"? I'm not sure why it would be happening today instead of on Sunday, but we're awfully close to the time change and at least that first test does make an assertion I would expect to break during the time change. |
| [09:13:58] | <symbioquine[m]> | ACTION uploaded an image: (249KiB) < https://matrix.org/oftc/media/v1/media/download/AQ7JGH5Vc5j4tTkf0dJpTYbC... > |
| [09:14:16] | <symbioquine[m]> | ACTION uploaded an image: (44KiB) < https://matrix.org/oftc/media/v1/media/download/AVeP-RXL0QbZkYRZJvVgahIZ... > |
| [09:15:02] | <symbioquine[m]> | I know it says line #59 not line #58 and also that it's being pretty explicit about what kind of assertion is failing... |
| [09:15:34] | <mstenta[m]> | Hmm interesting theory! |
| [09:15:58] | <mstenta[m]> | We have separate tests for the "reschedule" and "clone" actions, though... and the clone actions are the ones that are failing |
| [09:16:14] | <mstenta[m]> | (Maybe there's some spooky interaction though) |
| [09:16:25] | <mstenta[m]> | They all pass when I run them locally |
| [09:16:49] | <symbioquine[m]> | Can you re-run them on Drupal.org? |
| [09:16:52] | <mstenta[m]> | Just weird that it's saying two equal strings are not equal |
| [09:16:56] | <mstenta[m]> | Yep, I did a few times |
| [09:25:02] | <edbob[m]> | <mstenta[m]> "Maybe we should sketch up a..." <- morning - fyi the webhooks module wound up being good enough for what i need anyway. basically i'm done experimenting and ready to commit code. i'll post a lengthier "findings report" to the forum so we can follow up there if needed. |
| [09:25:50] | <edbob[m]> | also..sorry to hear that test failure is still a problem .. :/ the DST was a good idea/clue but i don't really have any to offer |
| [09:26:16] | <mstenta[m]> | great! |
| [11:10:28] | <mstenta[m]> | Ah ha! I figured out the testing issue... |
| [11:10:54] | <mstenta[m]> | The log names are links, but that doesn't show in the diff I pasted earlier... I found another diff that is the raw output: |
| [11:11:14] | <mstenta[m]> | ACTION sent a code block: https://matrix.org/oftc/media/v1/media/download/ASLS08b_Pu5wZbNhnxOXlgUr... |
| [11:11:17] | <mstenta[m]> | 🤦 |
| [11:11:23] | <mstenta[m]> | Drupal.org tests must run in a /web subdirectory. |
| [11:11:38] | <mstenta[m]> | So I just need to update my test to include the base path. |
| [13:49:04] | <mstenta[m]> | @room The farmOS monthly call starts in about 10 minutes. All are welcome! |
| [13:49:27] | <mstenta[m]> | This month's call will focus on the farmOS Conservation Planner module. |
| [13:50:05] | <mstenta[m]> | I'll be giving a quick demo of the module, and Greg will be talking about a market analysis that Our Sci did around it. |
| [13:50:14] | <mstenta[m]> | Here is the agenda: https://farmos.discourse.group/t/farmos-monthly-call-11-march-2026/2471 |
| [13:51:31] | <Greg[m]> | mstenta[m]: excited to get feedback on the Conservation Planner work - lots of info for folks! |