| [17:27:17] | <mstenta[m]> | <mstenta[m]> "paul121 symbioquine: Starting to..." <- symbioquine paul121 OK! I put together a PR that uninstalls `jsonapi_extras` and replicates the features we need in `farm_api` ... https://github.com/farmOS/farmOS/pull/964 |
| [17:28:21] | <mstenta[m]> | Note: This is for the 3.x branch! Not 4.x... I will open a separate 4.x-remove-jsonapi-extras branch that actually removes the module from composer.json. This one just uninstalls it (in case anyone else is depending on it). |
| [17:30:00] | <mstenta[m]> | I would love to get this merged ASAP, so that it can also be merged into 4.x, and then I can open the next PR. The PR for removing all update hooks will also be dependent on this, and that's a high priority for 4.x (it blocks any PRs that ADD update hooks to 4.x). |
| [17:31:24] | <mstenta[m]> | > Note: This is for the `3.x` branch! Not `4.x`... |
| [17:31:55] | <mstenta[m]> | I'm making an exception to the "everything needs to be against 4.x first, then backported" rule here 😅 but I think it makes sense in this instance |
| [17:56:26] | <paul121[m]> | <mstenta[m]> "I would love to get this..." <- LGTM, my only comment, which I just left on the PR, is if we could consider not adding a hook altogether |
| [17:56:38] | <paul121[m]> | I think we could do a simple yaml plugin instead |