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

deletionView and onSubmitSuccess/onSubmitError #1361

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tomy2105
Copy link

@tomy2105 tomy2105 commented Sep 5, 2017

This resolves #1106 and obsoletes (already closed) #1111.

admin-config pull request marmelab/admin-config#67 needed for this to function!

@tomy2105
Copy link
Author

tomy2105 commented Sep 6, 2017

Due to unfortunate usage of bash on windows at the moment (need some time to build my linux machine) I cannot run tests and thus unable to fix test case to follow my changes.
I've tested manually and my fix works, so I'm pretty sure it is only test case that needs update.
If anyone else might give me a hand with this I'd be grateful.

@tomy2105
Copy link
Author

tomy2105 commented Sep 8, 2017

OK, the initial error due to test case not being update is gone. But again I don't understand check failed error message "No selenium server jar found at", can someone please see what happened and why please.

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.

deletionView and onSubmitSuccess/onSubmitError
1 participant