Skip to content

Conversation

@tofran
Copy link
Member

@tofran tofran commented May 5, 2025

Load function is now:

load_function: fn _context, id ->
  get_user_by_id(id)
end

Also added :nil_is_not_found opt

@tofran tofran added the enhancement New feature or request label May 5, 2025
@tofran tofran requested review from Copilot and guilherme1805 May 5, 2025 10:51
@tofran tofran self-assigned this May 5, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • lib/middleware/arg_loader.ex: Language not supported

@tofran tofran changed the title Breaking: Add context to ArgLoader, add, improve errors and doc Breaking: Add context to ArgLoader function, nested keys feature and improve errors/doc May 5, 2025
@tofran tofran changed the title Breaking: Add context to ArgLoader function, nested keys feature and improve errors/doc Breaking: Add context to ArgLoader function, nil_is_not_found modifier, nested keys feature and improve errors/doc May 5, 2025
@tofran tofran requested a review from guilherme1805 May 5, 2025 13:55
@tofran tofran enabled auto-merge (squash) May 5, 2025 13:56
@tofran tofran disabled auto-merge May 5, 2025 14:08
@tofran tofran merged commit 413ca15 into main May 5, 2025
@tofran tofran deleted the arg-loader-improvements branch May 5, 2025 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

2 participants