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

FINERACT-881 #772

Merged
merged 1 commit into from Apr 19, 2020
Merged

FINERACT-881 #772

merged 1 commit into from Apr 19, 2020

Conversation

xurror
Copy link

@xurror xurror commented Apr 16, 2020

Remove all hard-coded default passwords

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

@vorburger
Copy link
Member

Following #764 and #761 this will pass the build (or not at least not fail because of integration tests) if you rebase this now.

Remove hard-coded default passwords
@xurror
Copy link
Author

xurror commented Apr 19, 2020

@vorburger, think this is good to go now

@vorburger
Copy link
Member

"I'm Lovin' this PR!" © ℗ ® ™

@vorburger vorburger merged commit 8b92b38 into apache:develop Apr 19, 2020
@vorburger
Copy link
Member

PS FTR I haven't actually pulled and myself locally re-verified (I did the original), but I'm sure you have tested this yourself on minikube while you made these changes, so I'll merge.

xurror pushed a commit to xurror/fineract that referenced this pull request Apr 19, 2020
@xurror xurror mentioned this pull request Apr 19, 2020
vorburger pushed a commit that referenced this pull request Apr 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants