PiccoloCRUD post_single return id of the inserted row instead of the row #255

Activity

1. sinisaos commented on Sep 20, 2023

   

   sinisaos

   on Sep 20, 2023

   Member

   More actions

   @JoshYuJump Right now you can't because of this line. If we change

   response = await row.save().run()
   json = dump_json(response)

   to this

   await row.save().run()
   json = dump_json(row.to_dict())

   we can get the response like this.

   {
       "id": 1,
       "name": "testing"
   }

   Another option is to not use PiccoloCRUD and write your own crud methods and use same to_dict() method.

   👍React with 👍1Reacted by Josh Yu
2. JoshYuJump commented on Sep 21, 2023

   

   JoshYuJump

   on Sep 21, 2023

   ContributorAuthor

   More actions

   @sinisaos
   I'm curious if you have any plans to do this in a future release.

   or it's a good practice right now?
3. sinisaos commented on Sep 21, 2023

   

   sinisaos

   on Sep 21, 2023

   Member

   More actions

   @JoshYuJump I think dict is a much better response than list, since we are POST single item. I can make a PR with these changes, but I'm not a maintainer and @dantownsend has to approve changes, if he agrees with these changes.
4. sinisaos

   closed this as a duplicateof CRUD Endpoint POST response is a list #211 on Jan 19, 2026