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

[RFR] Fix invalid reference to entry in datagrid header #708

Merged
merged 2 commits into from
Sep 29, 2015

Conversation

djhi
Copy link
Contributor

@djhi djhi commented Sep 25, 2015

  • Don't specify entry or headers
  • Fix blog example to check for entry before using it in getCssClasses for tag published
  • Updated documentation

@fzaninotto
Copy link
Member

I don't understand this PR. What is invalid?

@djhi
Copy link
Contributor Author

djhi commented Sep 25, 2015

ng-class="field.getCssClasses(entry)" entry does not exist at that point

@djhi
Copy link
Contributor Author

djhi commented Sep 25, 2015

And that was causing errors in the example app on the tags list view

@fzaninotto
Copy link
Member

Ah, you're right. But in that case, I suggest we keep the call, without passing entry as argument. This still allows to add a custom class to the header.

@djhi
Copy link
Contributor Author

djhi commented Sep 25, 2015

Ok but shouldn't we document that case ?

- [x] Don't specify `entry` or headers
- [x] Fix blog example to check for entry before using it in getCssClasses for tag published
- [x] Updated doc
fzaninotto added a commit that referenced this pull request Sep 29, 2015
[RFR] Fix invalid reference to entry in datagrid header
@fzaninotto fzaninotto merged commit 71f703e into master Sep 29, 2015
@fzaninotto fzaninotto deleted the fix_datagrid_header branch September 29, 2015 14:56
@fzaninotto
Copy link
Member

Thanks!

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.

2 participants