awasum commented on pull request #1286: URL: https://github.com/apache/fineract/pull/1286#issuecomment-680805549
> Removed string concatenated SQL and use of SQLseach parameter from Staff, > I have looked closely in what I am changing but a detailed review would help since we probably don't have good tests coverage on fineract yet. > > @awasum how do you think this as the project idea for outreachy this winter? WE NEED MORE TESTS!! Improving test overage will be a good Outreachy project. First thing, we need to identify scenarios which require unit or integration test on Fineract. Are there open JIRA issue for test coverage? If not, create one and begin preparing test cases and scenarios so that it will be easy for an intern to work on.. THat is if you are willing to mentor during the upcoming Outreachy session. @thesmallstar also engage on the Outreachy thread on the Dev list so we have all these ideas in one place and also create awareness as Outreachy and GSoC are two ways we get new contributors and new features.bug fixes on Fineract. CC @vorburger @ptuomola @xurror @percyashu ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
