Skip to content

[RFR] Delete button state-params fix#537

Closed
vasiakorobkin wants to merge 2 commits intomarmelab:masterfrom
vasiakorobkin:delete_button_fix
Closed

[RFR] Delete button state-params fix#537
vasiakorobkin wants to merge 2 commits intomarmelab:masterfrom
vasiakorobkin:delete_button_fix

Conversation

@vasiakorobkin
Copy link
Copy Markdown

Fixes #535

@jpetitcolas
Copy link
Copy Markdown
Contributor

Good to me. Can you just add a test to prove the bug fix? :)

@fzaninotto
Copy link
Copy Markdown
Member

No idea why the protractor build failed, I just restarted it.

@vasiakorobkin
Copy link
Copy Markdown
Author

Wrote end-to-end test to prove the bug fix.

@jpetitcolas
Copy link
Copy Markdown
Contributor

Needs rebase

@jpetitcolas jpetitcolas changed the title Delete button state-params fix [RFR] Delete button state-params fix Jul 2, 2015
@jpetitcolas
Copy link
Copy Markdown
Contributor

Can you prefix all your PR ready for review or merge by [RFR]? Would be easier to priorize our reviews. :)

@fzaninotto
Copy link
Copy Markdown
Member

still needs rebase...

@vasiakorobkin
Copy link
Copy Markdown
Author

Rebased branch and git push -f'ed it to vasiakorobkin/ng-admin.

@jpetitcolas
Copy link
Copy Markdown
Contributor

A last rebase to get a green tick from Travis? :)

@vasiakorobkin
Copy link
Copy Markdown
Author

Rebased. All test has been passed successfully.

@jpetitcolas
Copy link
Copy Markdown
Contributor

I merged another of your PR. Can you rebase a last time? :)

@vasiakorobkin
Copy link
Copy Markdown
Author

Done.

@jpetitcolas
Copy link
Copy Markdown
Contributor

Superseeded by #642.

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