-
Notifications
You must be signed in to change notification settings - Fork 134
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
Show an overlay and deleting message when deleting from a card or table action. #2415
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* v2-master: (29 commits) Fix console image name Fix not detecting admin user is admin scope is not stratos.admin Update status_updates.md Update passthrough.go Update roadmap Weekly status updates Remove auto select of service and improve service select Add delete space confirmation @ space level Removed console.log, fixed console.log linting, show warning for space pills as well Ignore cookie domain for now Fix file path for output Trigger on tag Update release pipeline Fix bad error formatting Tidy up pagination key creation - Ensure there's one common base way to create pagination key - Tidy up how that base is used WRT to entity relation pagination key - Ensure all lists that run against all endpoints have a common pagination key Fix space selection when user returns to CFOrgSpace selector Fix issue where circular reference failed to denormalize an app's space - Connect to scf, no other endpoint (bug is app specific) - to reproduce... - Fresh load on CF page - Click Services nav item - Click Applications nav item - apps in third column did not denormalize their space entity, causing undefined error - Caused by app's space's service instances containing app bindings Unit test fixes Fix unit tests Minor scss fix for code climate ...
Hey KlapTrap! Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA. |
Codecov Report
@@ Coverage Diff @@
## v2-master #2415 +/- ##
============================================
- Coverage 71.11% 71.1% -0.02%
============================================
Files 582 585 +3
Lines 24398 24514 +116
Branches 5467 5501 +34
============================================
+ Hits 17351 17430 +79
- Misses 7047 7084 +37 |
nwmac
changed the title
Show a overlay and deleting message when deleting from a card or table action.
Show an overlay and deleting message when deleting from a card or table action.
Jun 18, 2018
richard-cox
approved these changes
Jun 18, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See cf org list for an example.