-
Notifications
You must be signed in to change notification settings - Fork 99
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
Incompatibility with attribute grouping (Administrate 0.20.0) #65
Comments
Oh, dammit! Thanks for noticing that and opening this. If you could try with a PR, that'd be great! Let me know if you need any help. |
nickcharlton
added a commit
that referenced
this issue
Jan 25, 2024
0.20.0 introduces a change relating to attribute grouping and makes it incompatible. We'll pin this here and release a new version with this change so that a future breaking change can be marked as such. #65
nickcharlton
added a commit
that referenced
this issue
Jan 25, 2024
0.20.0 introduces a change relating to attribute grouping and makes it incompatible. We'll pin this here and release a new version with this change so that a future breaking change can be marked as such. #65
nickcharlton
pushed a commit
to trevorrjohn/administrate-field-nested_has_many
that referenced
this issue
Jan 25, 2024
In Administrate v0.20.0, support was added for grouping fields but this was not compatible with this field type. thoughtbot/administrate#2422 Fixes nickcharlton#65
trevorrjohn
added a commit
to trevorrjohn/administrate-field-nested_has_many
that referenced
this issue
Jan 26, 2024
In Administrate v0.20.0, support was added for grouping fields but this was not compatible with this field type. thoughtbot/administrate#2422 Fixes nickcharlton#65
We just updated to |
trevorrjohn
added a commit
to trevorrjohn/administrate-field-nested_has_many
that referenced
this issue
Feb 5, 2024
In Administrate v0.20.0, support was added for grouping fields but this was not compatible with this field type. thoughtbot/administrate#2422 Fixes nickcharlton#65
trevorrjohn
added a commit
to trevorrjohn/administrate-field-nested_has_many
that referenced
this issue
Feb 8, 2024
In Administrate v0.20.0, support was added for grouping fields but this was not compatible with this field type. thoughtbot/administrate#2422 Fixes nickcharlton#65
nickcharlton
pushed a commit
that referenced
this issue
Feb 14, 2024
In Administrate v0.20.0, support was added for grouping fields but this was not compatible with this field type. thoughtbot/administrate#2422 Fixes #65
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The latest version of Administrate groups attributes.
The return value for
Field#attributes
is now aHash
{"Group title" => [Field, Field]}
rather than anArray
[Field, Field]
. This is incompatible with the current version, throwingUndefined method 'attributes' for "":String (NoMethodError)
where""
is the group title.This is where the error is first raised:
I may be able to make a PR to handle the new format but in the meantime I wanted to document my findings.
The text was updated successfully, but these errors were encountered: