Skip to content

[#190][BUG] embedded_schema @primary_key false not supported #200

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

marinho10
Copy link

Closes #190 .

Fix error when in embedded_schema have @primary_key false

@marinho10
Copy link
Author

@rageofflames please see

type = Kaffy.ResourceSchema.field_type(changeset.data.__struct__, field)
build_html_input(changeset.data, form, {field, options}, type, opts)
end

defp build_html_input(schema, form, {field, options}, type, opts, readonly \\ false) do

Choose a reason for hiding this comment

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

@aesmail can you double check this chunk of code to see if it affects any logic?

@ghenry
Copy link
Collaborator

ghenry commented Apr 12, 2023

Hi,

Would you be so kind as to do this again to the master branch?

Thanks,
Gavin.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] embedded_schema @primary_key false not supported
3 participants