Mayaank Vadlamani
Mayaank Vadlamani
This is because the context is only available via socket. The `upsert_feedback` method is called via a rest endpoint in [server.py](https://github.com/Chainlit/chainlit/blob/4b533cd53173bcc24abe4341a7108f0070d60099/backend/chainlit/server.py#L679-L714). The [`FeedbackDict`](https://github.com/Chainlit/chainlit/blob/4b533cd53173bcc24abe4341a7108f0070d60099/backend/chainlit/types.py#L244) has a `forId` which is the id...
I think this is a great ask. As I noted in the [limitations](https://docs.chainlit.io/data-persistence/custom#limitations) section of the docs, this was explicitly not supported. The way dynamo works the data layer would...
Diffs LGTM. Next step is to get it past pipeline and for @willydouhard / @tpatel / team to test and approve.
No, I think the changes look good. Now it needs @willydouhard / teams approval
Are you able to make this fix @munday-tech?
Looks ok to me.