Skip to content

Resolves an issues preventing move working with keys: :atoms#27

Merged
corka149 merged 1 commit intocorka149:masterfrom
earobinson:master
Feb 25, 2025
Merged

Resolves an issues preventing move working with keys: :atoms#27
corka149 merged 1 commit intocorka149:masterfrom
earobinson:master

Conversation

@earobinson
Copy link
Copy Markdown
Contributor

Enables

move = %Jsonpatch.Operation.Move{from: "/a/b", path: "/a/e"}
target = %{a: %{b: %{c: "Bob"}}, d: false}
Jsonpatch.Operation.Move.apply(move, target, keys: :atoms)

To work, previously this would return an error.

@corka149 corka149 self-assigned this Feb 24, 2025
@corka149 corka149 added the bug Something isn't working label Feb 24, 2025
@corka149
Copy link
Copy Markdown
Owner

Awesome catch. Thx for the pr. I will soon release a new patch version.

@corka149 corka149 merged commit 768c05e into corka149:master Feb 25, 2025
10 checks passed
@corka149
Copy link
Copy Markdown
Owner

corka149 commented Mar 1, 2025

The bugfix was released with 2.2.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants