New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ecto 3.0 emits warnings when using arc_ecto #102
Comments
This warning is still occurring with It looks like this comment in arc_ecto may not be correct, and thus it's related code is not correct: # Cast supports both atom and string keys, ensure we're matching on both. |
@OvermindDL1 what's your changeset logic look like? I've been using Mine looks something like this: @required_fields ~w(first_name last_name)a
@optional_fields ~w(user_id company phone job_title linkedin_url job_description supervisor_name other)a
@doc false
def changeset(struct, attrs) do
struct
|> cast(attrs, @required_fields ++ @optional_fields)
|> ensure_uuid(:id)
|> cast_attachments(attrs, [:avatar, :other_file, :job_description_file])
|> ...
|
Figured out the reason, it was caused by having the file field in the |
When using Ecto 3.0 and arc_ecto, I see a lot of warnings about arc_ecto passing strings into
cast()
as the allowed list of param keys.Here are the ecto versions in my mix.lock
The text was updated successfully, but these errors were encountered: