add check for translating_date and comment actor check in the publish…

[mudongliang]

… operation

[inscripoem]

It seems that there is not a translating_date item in most source markdowns' metadata whose status is translating. Is this a new item? BTW, the corresponding comments may also needed to be updated regarding this function.

Besides, I think it would be better to add all other date order checks if there's already one cheking collected date and translating date in this pr. Apparently this will cost more duplicated lines extracting date items (like collected_date in this case, which is why I want to refactor the logic flow.

Of course the code itself is fine and ready to be merged. Paragraphs above are just for discussion.

[mudongliang]

It seems that there is not a translating_date item in most source markdowns' metadata whose status is translating. Is this a new item? BTW, the corresponding comments may also needed to be updated regarding this function.

Yes, for now there is no such a field. I would like to add this field in the frontmatter. And then keep this translating_date and translated_date, we can collect how many on average days usually one translator need.

Besides, I think it would be better to add all other date order checks if there's already one cheking collected date and translating date in this pr. Apparently this will cost more duplicated lines extracting date items (like collected_date in this case, which is why I want to refactor the logic flow.

I think the most functionability I would like to add is to identify if this PR changes from one status to another status. If we understand this point, it is really to check the integrity of the metadata. And this can avoid the FP of minor fixes after the proofreading or publishing.

Of course the code itself is fine and ready to be merged. Paragraphs above are just for discussion.