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] Add optional selection column to list #374

Merged
merged 24 commits into from
Apr 1, 2015
Merged

Conversation

ThieryMichel
Copy link
Contributor

ngadmin-batch-delete

@fzaninotto
Copy link
Member

Is it now RFR? Can you update the screenshot?

@fzaninotto
Copy link
Member

Don't display the action button if there are no selected items

entityName = this.entity.name();

this.DeleteQueries.batchDelete(this.view, this.entityIds).then(function () {
$location.path('/list/' + entityName);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use $state.go() instead, the url scheme may change in #376

Breaks the batchDelete page datagrid, to be fixed later
@fzaninotto
Copy link
Member

Saucelabs is drunk, tests are green on my side. I'm merging. Thanks for your work!

fzaninotto added a commit that referenced this pull request Apr 1, 2015
[RFR] Add optional selection column to list
@fzaninotto fzaninotto merged commit c04b5ca into master Apr 1, 2015
@fzaninotto fzaninotto deleted the multi_select_list branch April 1, 2015 20:22
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