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

refactor(*): de-angularize confirmationModalService #7759

Merged
merged 1 commit into from
Jan 10, 2020
Merged

refactor(*): de-angularize confirmationModalService #7759

merged 1 commit into from
Jan 10, 2020

Conversation

anotherchrisberry
Copy link
Contributor

Sorry for the size of this. There is not really a good way to break it up.

Copy link
Member

@erikmunson erikmunson left a comment

Choose a reason for hiding this comment

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

made it thru

outofbreath

@anotherchrisberry anotherchrisberry added the ready to merge Reviewed and ready for merge label Jan 10, 2020
@mergify mergify bot merged commit e6c6c66 into spinnaker:master Jan 10, 2020
@mergify mergify bot added the auto merged Merged automatically by a bot label Jan 10, 2020
@anotherchrisberry anotherchrisberry deleted the cm-de-ng branch January 10, 2020 01:26
jervi added a commit to jervi/deck that referenced this pull request Jan 31, 2020
b51dce4 chore(lint): Run eslint on typescript files
05c93f4 fix(core/kubernetes): clean up console noise in tests (spinnaker#7794)
e6c6c66 refactor(*): de-angularize confirmationModalService (spinnaker#7759)
eef40f6 fix(kubernetes): remove accidental static from resolveIndexedSecurityGroups (spinnaker#7760)
10b3491 refactor(*): use consistent styles on modal headers
a59b2c3 refactor(core): convert confirmation modal to react
6548872 refactor(core): provide wrapper for dangerously setting html (spinnaker#7721)
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Reviewed and ready for merge target-release/1.18
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants