Skip to content

Fix detecting dirty changes for :map fields#1003

Merged
andrykonchin merged 1 commit into
masterfrom
ak/fix-dirty-api-for-map-fields
Apr 19, 2026
Merged

Fix detecting dirty changes for :map fields#1003
andrykonchin merged 1 commit into
masterfrom
ak/fix-dirty-api-for-map-fields

Conversation

@andrykonchin
Copy link
Copy Markdown
Member

@andrykonchin andrykonchin commented Apr 19, 2026

Fixed an issue with map fields - just loaded model without any changes is reported as dirty.

Closes #1000

The issue was introduced in e2cfbab

@andrykonchin andrykonchin force-pushed the ak/fix-dirty-api-for-map-fields branch from 8024e55 to b05c92e Compare April 19, 2026 13:06
@andrykonchin andrykonchin force-pushed the ak/fix-dirty-api-for-map-fields branch from b05c92e to db2c257 Compare April 19, 2026 13:17
@github-actions
Copy link
Copy Markdown

Code Coverage

Package Line Rate Health
dynamoid 92%
Summary 92% (3826 / 4138)

Minimum allowed line rate is 90%

@andrykonchin andrykonchin merged commit 816e5e4 into master Apr 19, 2026
80 checks passed
@andrykonchin andrykonchin deleted the ak/fix-dirty-api-for-map-fields branch April 19, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3.11 broke Dirty + map due to symbols vs strings for keys (still present in 3.13)

1 participant