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

Batch Application context menu and delete refactor. #832

Merged
merged 9 commits into from
Nov 8, 2017

Conversation

ascobie
Copy link
Member

@ascobie ascobie commented Nov 6, 2017

Fix #776
Fix #831

  • Add context menu to batch application quick-list
  • Pass in enabled state to bl-delete-button
  • Delete action refactor to delete versions first
  • Disable delete multiple on quick-list if a selected application has allow updates to be false.

@ascobie
Copy link
Member Author

ascobie commented Nov 7, 2017

disabled quick-list delete as we cannot delete an application that have allowUpdates set to false and there is no easy way to check this and disable the quick-list delete.

Can be re-enabled in the future when i have time to look into a better solution, but can't see people wanting to bulk delete applications as we are limited to 20 anyway.

@ascobie ascobie merged commit 8de9d79 into master Nov 8, 2017
@ascobie ascobie deleted the feature/delete-app-package branch November 8, 2017 00:03
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.

1 participant