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

Add the shared and global properties resources to the bindings #1284

Conversation

rbotha78
Copy link
Contributor

The shared resources are required when working with CoralUI 3 multi-field values in HTL.

@update-changelog
Copy link

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update the CHANGELOG.md file with this pull request.

@coveralls
Copy link

coveralls commented Mar 19, 2018

Coverage Status

Coverage increased (+0.2%) to 35.315% when pulling 023d11e on rbotha78:feature/shared_props_resources into 61eff59 on Adobe-Consulting-Services:master.

@badvision
Copy link
Contributor

Please add test coverage for this new functionality (however small, I know...) -- a quick example is to write a until test that instantiates SharedComponentPropertiesBindingsValuesProvider and runs a few basic methods to ensure they are not returning null. This should allow your PR to pass the automated code coverage check

@rbotha78
Copy link
Contributor Author

Thanks @badvision
I've added a test for SharedComponentPropertiesBindingsValuesProvider 👍

@davidjgonzalez davidjgonzalez merged commit 2a0e28b into Adobe-Consulting-Services:master Mar 30, 2018
@rbotha78 rbotha78 deleted the feature/shared_props_resources branch February 15, 2019 15:53
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.

None yet

4 participants