search

zurichat / zc_messaging

19 stars 88 forks source link

feat: added endpoint to update a thread message #208

Closed Shopiley closed 1 year ago

Shopiley commented 1 year ago

Implemented update threads endpoint.

Note: This is a temporary solution as it uses an O(n) algorithm while we review a better way to implement threads (possibly as a collection of it's own). Also when testing the PUT /Update Message endpoint, the thread_id must be included in the request body.

@mikengr @zxenonx @AI-fae

FlorenceEgwu commented 1 year ago

Good Job Sope! I have left a few comments.

BTW, is there anything particularly stopping us from restructuring things and implementing this threads feature correctly right away?

Shopiley commented 1 year ago

Thanks for comments! I've made another PR implementing an improved solution. I also made corrections based on your comments here.