[Patchew-devel] [PATCH] rest: POST for message endpoint

Paolo Bonzini pbonzini at redhat.com
Mon May 7 12:22:48 UTC 2018


On 07/05/2018 13:19, Shubham Jain wrote:
> Alright. One more question.
> Setting in_reply_to here is the right thing to do for how the code
> behaves right now.  However, the real problem is that it's missing in
> MessageSerializer!  Can you add it, and here do:
> 
>     if 'in_reply_to' not in validated_data:
>         msg.in_reply_to = m.get_in_reply_to() or ""
> 
> Is this same for every field? msg.stripped_subject ,msg.version,
> msg.prefixes, msg.is_series_head ......

I had actually thought about that, and I think no.  Perhaps I should
have been more verbose in my original reply, perhaps it's better that
you thought about it and asked the question.

The reason why I think no, is that unlike in-reply-to those are
basically cached versions of data that is already elsewhere in the
Message object.

Paolo




More information about the Patchew-devel mailing list