search

Dexels / navajo

Navajo Service-oriented Applications
GNU Affero General Public License v3.0
9 stars 5 forks source link

fixed #577: taking message type into account when merging messages #578

Closed roelofkemp closed 3 years ago

roelofkemp commented 3 years ago

see: https://github.com/Dexels/navajo/issues/577

kharybdys commented 3 years ago

See comment in issue.

kharybdys commented 3 years ago

Also your fix should probably include a check on preferThis -> depending on its value prefer the one or the other submessage structure found

roelofkemp commented 3 years ago

Discussed with Marte, we prefer to throw when incompatible messages are merged.

ghost commented 3 years ago

Approved based on limited knowledge.