Dev 1220 backend integration examples laravel symfony nest js express with docker compose#33
Conversation
…xpress with Docker Compose
…xpress with Docker Compose
…xpress with Docker Compose
✅ Deploy Preview for handsontable-examples canceled.
|
✅ Deploy Preview for handsontable-examples canceled.
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
| title: 'Row added', | ||
| message: `Created ${data.length} row${data.length !== 1 ? 's' : ''}`, | ||
| duration: 3000, | ||
| }); |
There was a problem hiding this comment.
Frontend crashes accessing .length on null response
High Severity
The store method in ProductController.php returns response()->json(null, 201), meaning res.json() in the frontend resolves to null. The onRowsCreate callback then accesses data.length on this null value, causing a TypeError at runtime every time a row is created.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 35f88a8. Configure here.
…xpress with Docker Compose
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit b375237. Configure here.
| title: 'Row added', | ||
| message: `Created: ${row.sku} (id: ${row.id})`, | ||
| duration: 3000, | ||
| }); |
There was a problem hiding this comment.
Missing return value in Laravel onRowsCreate callback
High Severity
The onRowsCreate handler in the Laravel frontend reads the server response (which now includes the created products with server-assigned IDs) and shows a notification, but never returns the data. Both the Django and NestJS examples correctly return data so the dataProvider plugin can map server-assigned IDs to the newly created rows. Without this return, the dataProvider won't know the real IDs of created products, causing subsequent update or delete operations on those rows to fail.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit b375237. Configure here.
2 participants
Backend integration examples — Django, Laravel, NestJS, Rails, Spring, Symfony with Docker Compose
Note
Medium Risk
Moderate risk because it changes the request/response contracts for create/update operations and adds a new ordering column/migration in the Laravel example, which can break existing clients or require a DB reset for demos.
Overview
Updates the server-side integration examples to match newer Handsontable
dataProviderpayload shapes and improve UX around row mutations. Django now creates rows based onrowsAmountand updates rows viarow['changes'], while the Django/NestJS/Laravel frontends add notifications and an in-grid delete confirmation viabeforeRowsMutation.Improves determinism and insertion behavior in the Laravel example. Adds a
sort_ordercolumn (migration + model + seeding), defaults list sorting tosort_order, and updatesPOST /api/productsto insert above/below a reference row by shiftingsort_orderin a DB transaction and returning created rows.Polish and tooling tweaks. Adds favicons, pins example frontends to
handsontable: experimental(with new lockfiles), simplifies docker-compose files by removing explicitversion, and expands the NestJSMakefile/README withstart/stop/logs/clean/reset/psqltargets plus a Vite proxy for/tickets.Reviewed by Cursor Bugbot for commit fbf43c0. Bugbot is set up for automated code reviews on this repo. Configure here.