ecamp / ecamp/ecamp3

Consider moving JsonMergePatch::mergePatch into a deserializer

Open
#2,947 0 comments 0 reactions 0 assignees View on GitHub
agents:ignore
Dominant language
PHP
Stars
156
Forks
72
Avg merge
12h 43m
Merged PRs (30d)
203

Description

Hätten wir hier irgendwie die Möglichkeit, auch mal JsonPatch zu implementieren, und dann je nach ContentType im Request eine andere Merge-Strategie zu verwenden? Bei dieser Frage bekomme ich irgendwie das Gefühl dass das hier eher in einen Denormalizer gehört. Wäre das eine Möglichkeit?

Wenn du das im Setter machst wirds auch z.B. schwieriger, Tests zu schreiben etc.

_Originally posted by @carlobeltrame in https://github.com/ecamp/ecamp3/pull/2825#discussion_r955138200_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.