search

westonganger / active_snapshot

Simplified snapshots and restoration for ActiveRecord models and associations with a transparent white-box implementation
MIT License
103 stars 16 forks source link

Make first argument of create_snapshot a named argument #26

Closed westonganger closed 1 year ago

westonganger commented 1 year ago

Lets change the identifier option from a positional argument to a named argument.

Current docs

snapshot = post.create_snapshot!(
  "snapshot_1",
  user: current_user,
  metadata: {},
)

Updated

snapshot = post.create_snapshot!(
  identifier: "snapshot_1",
  user: current_user, # optional
  metadata: {},
)
westonganger commented 1 year ago

Merged to master on Sept 19 2022, will update this issue once a new version is released.