Skip to content

Conversation

greg-rychlewski
Copy link
Member

I wasn't 100% confident this is what you were thinking but please have a look!

@josevalim
Copy link
Member

Thank you @greg-rychlewski! This is exactly what I had in mind but I actually did not remember we already supported as(:posts). Given that, perhaps we only need to support as(^atom) (perhaps we already do) and update Ecto.Query.API.field/2 docs?

@greg-rychlewski
Copy link
Member Author

@josevalim We support both. Updated docs :)

Copy link
Member

@josevalim josevalim left a comment

Choose a reason for hiding this comment

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

Thank you and sorry for sending you down the wrong rabbit hole!

@greg-rychlewski greg-rychlewski merged commit ab64e1d into elixir-ecto:master Sep 27, 2025
7 checks passed
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