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 new "var" property to taglib #68

Merged
merged 1 commit into from Jan 17, 2014
Merged

Add new "var" property to taglib #68

merged 1 commit into from Jan 17, 2014

Conversation

@ensonik
Copy link
Contributor

@ensonik ensonik commented Jan 16, 2014

The new var property gives the taglib the capability to store the state
of a feature in a page context variable. This is useful when you want
to use the result in EL expressions (amongst other uses).

No unit tests were written. I tried. I tried hard. But configuring the arquillian container on my setup proved to be above my IQ levels.

The new var property gives the taglib the capability to store the state
of a feature in a page context variable. This is useful when you want
to use the result in EL expressions (amongst other uses)
@chkal
Copy link
Member

@chkal chkal commented Jan 17, 2014

Hey @ensonik.

Thanks a lot for your pull request. Your patch looks good and I think this new feature will be useful for other.

Christian

chkal added a commit that referenced this pull request Jan 17, 2014
Add new "var" property to taglib
@chkal chkal merged commit 36a5a77 into togglz:master Jan 17, 2014
1 check passed
1 check passed
default The Travis CI build passed
Details
@ensonik
Copy link
Contributor Author

@ensonik ensonik commented Jan 17, 2014

Great news for us. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.