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

Show an overlay and deleting message when deleting from a card or table action. #2415

Merged
merged 11 commits into from
Jun 18, 2018

Conversation

KlapTrap
Copy link
Contributor

See cf org list for an example.

KlapTrap and others added 9 commits June 14, 2018 11:00
* 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
  ...
@cfdreddbot
Copy link

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.

@KlapTrap KlapTrap self-assigned this Jun 18, 2018
@codecov
Copy link

codecov bot commented Jun 18, 2018

Codecov Report

Merging #2415 into v2-master will decrease coverage by 0.01%.
The diff coverage is 72.05%.

@@             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 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 richard-cox self-requested a review June 18, 2018 14:38
@richard-cox richard-cox merged commit d547a8c into v2-master Jun 18, 2018
@richard-cox richard-cox deleted the table-card-deleting branch June 18, 2018 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants