Skip to content

Commit 1fade1e

Browse files
authored
Do not require a "display" string in Member (#75)
Fixes #73
1 parent ca323f5 commit 1fade1e

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

core/src/patch.rs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,6 @@ fn apply_group_add_op(
277277
#[derive(Debug, Deserialize)]
278278
struct Member {
279279
value: String,
280-
display: String,
281280
}
282281
let Ok(member) = serde_json::from_value::<Member>(member.clone())
283282
else {
@@ -286,13 +285,6 @@ fn apply_group_add_op(
286285
));
287286
};
288287

289-
if group.resource.display_name.eq_ignore_ascii_case(&member.display) {
290-
return Err(PatchRequestError::Invalid(format!(
291-
"group add op value display was {} expected {}",
292-
member.display, group.resource.display_name
293-
)));
294-
}
295-
296288
// 3.5.2.1. Add Operation
297289
//
298290
// If the target location already contains the value specified, no

0 commit comments

Comments
 (0)