Skip to content

Conversation

@aadamcik
Copy link
Contributor

❓ Type of change

  • πŸ‘Œ Enhancement (improving an existing functionality like performance)

πŸ“š Description

This PR adds isDirty(fields) method for easier checking whether field has changed. Which could be replacement for .$isDirty

// isDirty(fields?: keyof ModelAttributes<this> | (keyof ModelAttributes<this>)[]): boolean

const user = new User()
user.username = 'test'

user.isDirty('username') // true
user.isDirty(['username']) // true
user.isDirty() // true

Which is easier to type and type-safe vs 'username' in user.$dirty and more convenient when checking multiple fields, if any is dirty.

@thetutlage thetutlage merged commit 151f862 into adonisjs:21.x Dec 4, 2024
16 checks passed
@thetutlage
Copy link
Member

Thanks πŸ‘

nadlgit pushed a commit to nadlgit/adonisjs-lucid that referenced this pull request Jun 28, 2025
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.

2 participants