Skip to content
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

Support element initialization via components attributes Hash #54

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

Conversation

msalzburg
Copy link
Contributor

@msalzburg msalzburg commented Aug 9, 2023

Was able to support element initialization via components attributes Hash by:

  • Adding tests for error cases
  • Renaming the components attributes Hash to attributes_or_elements.
  • Enhancing the initialize_elements method to lookup available element
    values from the attributes_or_elements Hash and assigning them.

Fixes #53 and #51 (this PR superseeds PR #52)

See relevant changes in a71fe1b

Any thoughts? Is a timely merge possible?

Looking forward!

* Update github actions
* Setup build matrix with ruby versions between 2.5.9 and 3.2.2
* Restrict gem to rails versions >= 5.1 and < 7.0
* Update development dependencies
Renames the components attributes Hash to attributes_or_elements.
Enhances the initialize_elements method to lookup available element
values from the attributes_or_elements Hash and assigns them.
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.

Element initialization via components attributes Hash is not working
1 participant