fineract-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [fineract] awasum commented on pull request #1286: Removed string concatenated SQL & sqlSearch from Staff (Fineract-854)
Date Wed, 26 Aug 2020 10:48:48 GMT

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:
users@infra.apache.org



Mime
View raw message