[GitHub] [fineract] vidakovic merged pull request #1973: FINERACT-1433: Switch to Java based config for Spring

2021-11-20 Thread GitBox
vidakovic merged pull request #1973: URL: https://github.com/apache/fineract/pull/1973 -- 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. To unsubscribe, e-mail: commits-uns

[GitHub] [fineract] ptuomola opened a new pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-21 Thread GitBox
ptuomola opened a new pull request #1975: URL: https://github.com/apache/fineract/pull/1975 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/p

[GitHub] [fineract] vidakovic merged pull request #1973: FINERACT-1433: Switch to Java based config for Spring

2021-11-21 Thread GitBox
vidakovic merged pull request #1973: URL: https://github.com/apache/fineract/pull/1973 -- 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. To unsubscribe, e-mail: commits-uns

[GitHub] [fineract] BLasan commented on pull request #1933: Feat: Filter transactions from last interest posting date

2021-11-21 Thread GitBox
BLasan commented on pull request #1933: URL: https://github.com/apache/fineract/pull/1933#issuecomment-974642152 @avikganguly01 @fynmanoj Please review -- 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

[GitHub] [fineract] rrpawar96 opened a new pull request #1976: FINERACT-1053:CustomerAccountNumber

2021-11-23 Thread GitBox
rrpawar96 opened a new pull request #1976: URL: https://github.com/apache/fineract/pull/1976 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/

[GitHub] [fineract] rrpawar96 closed pull request #1976: FINERACT-1053:CustomerAccountNumber

2021-11-23 Thread GitBox
rrpawar96 closed pull request #1976: URL: https://github.com/apache/fineract/pull/1976 -- 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. To unsubscribe, e-mail: commits-uns

[GitHub] [fineract] github-actions[bot] commented on pull request #1852: chore(deps): update plugin org.openapi.generator to v5

2021-11-23 Thread GitBox
github-actions[bot] commented on pull request #1852: URL: https://github.com/apache/fineract/pull/1852#issuecomment-977305449 This pull request seems to be stale. Are you still planning to work on it? We will automatically close it in 30 days. -- This is an automated message from the A

[GitHub] [fineract] ptuomola opened a new pull request #1977: Fixing CORS issue introduced by Spring Boot upgrade

2021-11-23 Thread GitBox
ptuomola opened a new pull request #1977: URL: https://github.com/apache/fineract/pull/1977 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/p

[GitHub] [fineract] ptuomola closed pull request #1977: Fixing CORS issue introduced by Spring Boot upgrade

2021-11-23 Thread GitBox
ptuomola closed pull request #1977: URL: https://github.com/apache/fineract/pull/1977 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] ptuomola opened a new pull request #1978: Fixing CORS issue introduced by Spring Boot upgrade

2021-11-23 Thread GitBox
ptuomola opened a new pull request #1978: URL: https://github.com/apache/fineract/pull/1978 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/p

[GitHub] [fineract] ptuomola commented on pull request #1978: Fixing CORS issue introduced by Spring Boot upgrade

2021-11-23 Thread GitBox
ptuomola commented on pull request #1978: URL: https://github.com/apache/fineract/pull/1978#issuecomment-977618840 @rrpawar96 could you check if this PR fixes your CORS error issue? If yes, I can merge it in given the build passes. -- This is an automated message from the Apache Git Ser

[GitHub] [fineract] rrpawar96 commented on pull request #1978: Fixing CORS issue introduced by Spring Boot upgrade

2021-11-24 Thread GitBox
rrpawar96 commented on pull request #1978: URL: https://github.com/apache/fineract/pull/1978#issuecomment-977697287 @ptuomola, Yes tested, it's working all good. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [fineract] rrpawar96 opened a new pull request #1979: FINERACT-1375:ClientEntityPerson

2021-11-24 Thread GitBox
rrpawar96 opened a new pull request #1979: URL: https://github.com/apache/fineract/pull/1979 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/

[GitHub] [fineract] ptuomola merged pull request #1978: Fixing CORS issue introduced by Spring Boot upgrade

2021-11-24 Thread GitBox
ptuomola merged pull request #1978: URL: https://github.com/apache/fineract/pull/1978 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] ptuomola commented on pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-24 Thread GitBox
ptuomola commented on pull request #1975: URL: https://github.com/apache/fineract/pull/1975#issuecomment-977764933 @vidakovic This upgrades the Spring Boot to 2.6.0 - an upgrade that was unblocked by the config file rework. If you can approve and merge this, as next step I'll then rework t

[GitHub] [fineract] vidakovic commented on a change in pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-24 Thread GitBox
vidakovic commented on a change in pull request #1975: URL: https://github.com/apache/fineract/pull/1975#discussion_r756009170 ## File path: fineract-provider/src/main/java/org/apache/fineract/infrastructure/security/filter/TenantAwareBasicAuthenticationFilter.java ## @@ -59,3

[GitHub] [fineract] vidakovic commented on pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-24 Thread GitBox
vidakovic commented on pull request #1975: URL: https://github.com/apache/fineract/pull/1975#issuecomment-977826524 Overall looks good to me. Just one caveat: at the moment there is no compatible Spring Cloud release for Spring Boot 2.6.0. Just in case someone is using that combination in

[GitHub] [fineract] vidakovic edited a comment on pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-24 Thread GitBox
vidakovic edited a comment on pull request #1975: URL: https://github.com/apache/fineract/pull/1975#issuecomment-977826524 Overall looks good to me. Just one caveat: at the moment there is no compatible Spring Cloud release for Spring Boot 2.6.0. Just in case someone is using that combinat

[GitHub] [fineract] ptuomola commented on pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-24 Thread GitBox
ptuomola commented on pull request #1975: URL: https://github.com/apache/fineract/pull/1975#issuecomment-977842201 > Overall looks good to me. Just one caveat: at the moment there is no compatible Spring Cloud release for Spring Boot 2.6.0. Just in case someone is using that combination in

[GitHub] [fineract] ptuomola commented on a change in pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-24 Thread GitBox
ptuomola commented on a change in pull request #1975: URL: https://github.com/apache/fineract/pull/1975#discussion_r756027492 ## File path: fineract-provider/src/main/java/org/apache/fineract/infrastructure/security/filter/TenantAwareBasicAuthenticationFilter.java ## @@ -59,33

[GitHub] [fineract] ptuomola opened a new pull request #1980: FINERACT-1089: Fixing 'Operation not allowed for a result set of type"

2021-11-24 Thread GitBox
ptuomola opened a new pull request #1980: URL: https://github.com/apache/fineract/pull/1980 ## Description Fix as proposed by Victor Jo in the JIRA ticket ## Checklist Please make sure these boxes are checked before submitting your pull request - thanks! - [

[GitHub] [fineract] ptuomola opened a new pull request #1981: Only use Travis for SonarQube

2021-11-24 Thread GitBox
ptuomola opened a new pull request #1981: URL: https://github.com/apache/fineract/pull/1981 ## Description As we now have all the integration and docker tests running in a Github Action, we can disable the same activities from Travis. We need to use Travis for SonarQube as there is

[GitHub] [fineract] francisguchie commented on pull request #1978: Fixing CORS issue introduced by Spring Boot upgrade

2021-11-24 Thread GitBox
francisguchie commented on pull request #1978: URL: https://github.com/apache/fineract/pull/1978#issuecomment-977903376 @rrpawar96 and @ptuomola thank you for this good work -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [fineract] ptuomola closed pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-24 Thread GitBox
ptuomola closed pull request #1975: URL: https://github.com/apache/fineract/pull/1975 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] vidakovic merged pull request #1975: Upgrade to Spring Boot 2.6.0

2021-11-24 Thread GitBox
vidakovic merged pull request #1975: URL: https://github.com/apache/fineract/pull/1975 -- 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. To unsubscribe, e-mail: commits-uns

[GitHub] [fineract] avikganguly01 merged pull request #1933: Feat: Filter transactions from last interest posting date

2021-11-24 Thread GitBox
avikganguly01 merged pull request #1933: URL: https://github.com/apache/fineract/pull/1933 -- 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. To unsubscribe, e-mail: commits

[GitHub] [fineract] ptuomola closed pull request #1980: FINERACT-1089: Fixing 'Operation not allowed for a result set of type"

2021-11-24 Thread GitBox
ptuomola closed pull request #1980: URL: https://github.com/apache/fineract/pull/1980 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] rrpawar96 opened a new pull request #1982: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-25 Thread GitBox
rrpawar96 opened a new pull request #1982: URL: https://github.com/apache/fineract/pull/1982 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/

[GitHub] [fineract] rrpawar96 closed pull request #1982: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-25 Thread GitBox
rrpawar96 closed pull request #1982: URL: https://github.com/apache/fineract/pull/1982 -- 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. To unsubscribe, e-mail: commits-uns

[GitHub] [fineract] IOhacker commented on pull request #1891: Updated Java to Azul Zulu jdk17

2021-11-25 Thread GitBox
IOhacker commented on pull request #1891: URL: https://github.com/apache/fineract/pull/1891#issuecomment-979280078 I will close this PR, because I have tested the latest update of Spring 2.6.0 and there are less issues. I will submit another PR -- This is an automated message from the Ap

[GitHub] [fineract] IOhacker closed pull request #1891: Updated Java to Azul Zulu jdk17

2021-11-25 Thread GitBox
IOhacker closed pull request #1891: URL: https://github.com/apache/fineract/pull/1891 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] IOhacker opened a new pull request #1983: Upgrade JDK 11 LTS to JDK 17 LTS (FINERACT-1406)

2021-11-25 Thread GitBox
IOhacker opened a new pull request #1983: URL: https://github.com/apache/fineract/pull/1983 ## Description Java has been upgraded from v11 LTS to v17 LTS https://paste.apache.org/nd32y ## Checklist Please make sure these boxes are checked before submitting your pu

[GitHub] [fineract] francisguchie commented on pull request #1979: FINERACT-1375:ClientEntityPerson

2021-11-25 Thread GitBox
francisguchie commented on pull request #1979: URL: https://github.com/apache/fineract/pull/1979#issuecomment-979337940 Thanks @rrpawar96 @awasum @ptuomola please note that i have already tested the functionality of this and works for me like -- This is an automated message from

[GitHub] [fineract] francisguchie commented on pull request #1976: FINERACT-1053:CustomerAccountNumber

2021-11-25 Thread GitBox
francisguchie commented on pull request #1976: URL: https://github.com/apache/fineract/pull/1976#issuecomment-979338917 @rrpawar96, @awasum @ptuomola I have tested this and already have it in live production -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [fineract] francisguchie commented on pull request #1982: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-25 Thread GitBox
francisguchie commented on pull request #1982: URL: https://github.com/apache/fineract/pull/1982#issuecomment-979340712 @rrpawar96 @awasum @ptuomola - this functionality has already been tested by using a branch from @rrpawar96 's repo. It is ok with me. -- This is an automated mess

[GitHub] [fineract] rrpawar96 opened a new pull request #1984: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-25 Thread GitBox
rrpawar96 opened a new pull request #1984: URL: https://github.com/apache/fineract/pull/1984 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/

[GitHub] [fineract] francisguchie commented on pull request #1984: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-25 Thread GitBox
francisguchie commented on pull request #1984: URL: https://github.com/apache/fineract/pull/1984#issuecomment-979448970 @rrpawar96 @ptuomola @awasum this i have tested and works for me -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [fineract] github-actions[bot] commented on pull request #1725: Create Client doesn't giving error on submit without filling form #3069

2021-11-25 Thread GitBox
github-actions[bot] commented on pull request #1725: URL: https://github.com/apache/fineract/pull/1725#issuecomment-979525505 This pull request seems to be stale. Are you still planning to work on it? We will automatically close it in 30 days. -- This is an automated message from the A

[GitHub] [fineract] vidakovic merged pull request #1983: Upgrade JDK 11 LTS to JDK 17 LTS (FINERACT-1406)

2021-11-25 Thread GitBox
vidakovic merged pull request #1983: URL: https://github.com/apache/fineract/pull/1983 -- 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. To unsubscribe, e-mail: commits-uns

[GitHub] [fineract] francisguchie commented on pull request #1976: FINERACT-1053:CustomerAccountNumber

2021-11-26 Thread GitBox
francisguchie commented on pull request #1976: URL: https://github.com/apache/fineract/pull/1976#issuecomment-979882565 @rrpawar96 i think the title should read FINERACT-1053:CustomAccountNumber (custom) -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [fineract] rrpawar96 opened a new pull request #1985: 1391-AccountNumberPreferencesPrefixEight

2021-11-26 Thread GitBox
rrpawar96 opened a new pull request #1985: URL: https://github.com/apache/fineract/pull/1985 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/

[GitHub] [fineract] bharathcgowda commented on pull request #1985: FINERACT-1391:AccountNumberPreferencesPrefixEight

2021-11-26 Thread GitBox
bharathcgowda commented on pull request #1985: URL: https://github.com/apache/fineract/pull/1985#issuecomment-980031678 @rrpawar96 thank you for sending this fix, could you please check and fix the travis issue? @ptuomola @vidakovic @thesmallstar It would be really helpful if any

[GitHub] [fineract] ptuomola merged pull request #1981: Only use Travis for SonarQube

2021-11-26 Thread GitBox
ptuomola merged pull request #1981: URL: https://github.com/apache/fineract/pull/1981 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] ptuomola merged pull request #1976: FINERACT-1053:CustomAccountNumber

2021-11-26 Thread GitBox
ptuomola merged pull request #1976: URL: https://github.com/apache/fineract/pull/1976 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] vidakovic commented on pull request #1985: FINERACT-1391:AccountNumberPreferencesPrefixEight

2021-11-26 Thread GitBox
vidakovic commented on pull request #1985: URL: https://github.com/apache/fineract/pull/1985#issuecomment-980153939 btw: I have a fix for the Travis issue... will submit later -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [fineract] rrpawar96 commented on pull request #1985: FINERACT-1391:AccountNumberPreferencesPrefixEight

2021-11-26 Thread GitBox
rrpawar96 commented on pull request #1985: URL: https://github.com/apache/fineract/pull/1985#issuecomment-980177932 Yes, it seems the Travis build is been fixed by this PR #1981, let me also try again by rebasing. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [fineract] ptuomola closed pull request #1725: Create Client doesn't giving error on submit without filling form #3069

2021-11-26 Thread GitBox
ptuomola closed pull request #1725: URL: https://github.com/apache/fineract/pull/1725 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] ptuomola commented on pull request #1725: Create Client doesn't giving error on submit without filling form #3069

2021-11-26 Thread GitBox
ptuomola commented on pull request #1725: URL: https://github.com/apache/fineract/pull/1725#issuecomment-980499067 Let's see if the build passes now... -- 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

[GitHub] [fineract] ptuomola commented on pull request #1725: Create Client doesn't giving error on submit without filling form #3069

2021-11-26 Thread GitBox
ptuomola commented on pull request #1725: URL: https://github.com/apache/fineract/pull/1725#issuecomment-980503370 This change makes half of the integration tests to fail... that needs to be investigated / fixed before merging -- This is an automated message from the Apache Git Service.

[GitHub] [fineract] ptuomola commented on pull request #1883: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

2021-11-26 Thread GitBox
ptuomola commented on pull request #1883: URL: https://github.com/apache/fineract/pull/1883#issuecomment-980516413 Hi @IOhacker - will you have time to finish this one off? I can give it a go if you're busy - would be great to get this implemented... -- This is an automated message from

[GitHub] [fineract] vidakovic opened a new pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic opened a new pull request #1986: URL: https://github.com/apache/fineract/pull/1986 ## Description - proper Docker image build with reasonable startup parameters (the ones that were in place until now were not really great) - using the latest JDK 17 LTS release everywhere

[GitHub] [fineract] ptuomola commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
ptuomola commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757832128 ## File path: .travis.yml ## @@ -57,13 +48,8 @@ install: - curl -O https://cdn.azul.com/zulu/bin/zulu-repo_1.0.0-3_all.deb - sudo apt-get -yq inst

[GitHub] [fineract] vidakovic commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757833279 ## File path: .travis.yml ## @@ -57,13 +48,8 @@ install: - curl -O https://cdn.azul.com/zulu/bin/zulu-repo_1.0.0-3_all.deb - sudo apt-get -yq ins

[GitHub] [fineract] vidakovic commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757833593 ## File path: fineract-provider/build.gradle ## @@ -274,10 +252,50 @@ bootJar { classifier = '' } -bootWar { -enabled = false +jib { +f

[GitHub] [fineract] vidakovic commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757833811 ## File path: fineract-provider/build.gradle ## @@ -289,7 +307,6 @@ distributions { distributionBaseName = "apache-fineract-binary" c

[GitHub] [fineract] ptuomola commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
ptuomola commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757834318 ## File path: fineract-provider/build.gradle ## @@ -274,10 +252,50 @@ bootJar { classifier = '' } -bootWar { -enabled = false +jib { +fr

[GitHub] [fineract] vidakovic commented on pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic commented on pull request #1986: URL: https://github.com/apache/fineract/pull/1986#issuecomment-980826177 ... not sure if my comment made it to GH... so here again: We can do that (environment variables to configure basic auth vs. oauth)... could we just do this in another

[GitHub] [fineract] vidakovic commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757835136 ## File path: .travis.yml ## @@ -57,13 +48,8 @@ install: - curl -O https://cdn.azul.com/zulu/bin/zulu-repo_1.0.0-3_all.deb - sudo apt-get -yq ins

[GitHub] [fineract] vidakovic commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757836960 ## File path: fineract-provider/build.gradle ## @@ -274,10 +252,50 @@ bootJar { classifier = '' } -bootWar { -enabled = false +jib { +f

[GitHub] [fineract] vidakovic commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757837024 ## File path: fineract-provider/build.gradle ## @@ -289,7 +307,6 @@ distributions { distributionBaseName = "apache-fineract-binary" c

[GitHub] [fineract] vidakovic commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
vidakovic commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757838707 ## File path: fineract-provider/build.gradle ## @@ -289,7 +307,6 @@ distributions { distributionBaseName = "apache-fineract-binary" c

[GitHub] [fineract] ptuomola commented on a change in pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-27 Thread GitBox
ptuomola commented on a change in pull request #1986: URL: https://github.com/apache/fineract/pull/1986#discussion_r757840732 ## File path: fineract-provider/build.gradle ## @@ -289,7 +307,6 @@ distributions { distributionBaseName = "apache-fineract-binary" co

[GitHub] [fineract] ptuomola opened a new pull request #1987: Implementing automatic inclusion of licenses

2021-11-28 Thread GitBox
ptuomola opened a new pull request #1987: URL: https://github.com/apache/fineract/pull/1987 ## Description The licenses of 3rd party packages we were including in the distribution packages were rather out-of-date. Instead of "hardcoding" the licenses - which would require a l

[GitHub] [fineract] Ankit-muellner commented on pull request #1987: Automatic inclusion of 3rd party licenses in the distribution package

2021-11-28 Thread GitBox
Ankit-muellner commented on pull request #1987: URL: https://github.com/apache/fineract/pull/1987#issuecomment-981067967 that's a good strategy. thanks @ptuomola -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [fineract] ptuomola opened a new pull request #1988: Changing logging level for findbugs

2021-11-28 Thread GitBox
ptuomola opened a new pull request #1988: URL: https://github.com/apache/fineract/pull/1988 ## Description Findbugs logs so much output that it is impossible to find the test results in the mix. This disables the progress reporting from Findbugs. ## Checklist Pleas

[GitHub] [fineract] vidakovic commented on pull request #1988: Changing logging level for findbugs

2021-11-28 Thread GitBox
vidakovic commented on pull request #1988: URL: https://github.com/apache/fineract/pull/1988#issuecomment-981119421 @ptuomola thank you... that one was really annoying and polluting the build logs... -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [fineract] vidakovic merged pull request #1988: Changing logging level for findbugs

2021-11-28 Thread GitBox
vidakovic merged pull request #1988: URL: https://github.com/apache/fineract/pull/1988 -- 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. To unsubscribe, e-mail: commits-uns

[GitHub] [fineract] vidakovic commented on pull request #1986: FINERACT-1446: Docker image build with Jib and JDK 17 fixes

2021-11-28 Thread GitBox
vidakovic commented on pull request #1986: URL: https://github.com/apache/fineract/pull/1986#issuecomment-981351366 @ptuomola ... well... step by step sometimes doesn't work. Docker Compose on Github Actions is just not working, so I replaced the integration test setup as well (which was a

[GitHub] [fineract] fynmanoj commented on a change in pull request #1984: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-29 Thread GitBox
fynmanoj commented on a change in pull request #1984: URL: https://github.com/apache/fineract/pull/1984#discussion_r758322188 ## File path: fineract-provider/src/main/java/org/apache/fineract/portfolio/client/domain/ClientIdentifier.java ## @@ -58,20 +62,29 @@ private Int

[GitHub] [fineract] fynmanoj commented on pull request #1984: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-29 Thread GitBox
fynmanoj commented on pull request #1984: URL: https://github.com/apache/fineract/pull/1984#issuecomment-981599347 @rrpawar96 , @francisguchie as per the Screenshots attached to JIRA ticket the new field appears to be a dropdown. where does the dropdown values are fetched from? Also if the

[GitHub] [fineract] sheminanto opened a new pull request #1989: FINERACT-1447 : Fix - Overdraft interest calculated is wrong

2021-11-29 Thread GitBox
sheminanto opened a new pull request #1989: URL: https://github.com/apache/fineract/pull/1989 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract

[GitHub] [fineract] ptuomola opened a new pull request #1990: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

2021-11-29 Thread GitBox
ptuomola opened a new pull request #1990: URL: https://github.com/apache/fineract/pull/1990 ## Description Building on (and including) the excellent work of @IOhacker, this completes the removal of Drizzle from all parts of the code. This also stops including any JDBC driver in the

[GitHub] [fineract] ptuomola merged pull request #1989: FINERACT-1447 : Fix - Overdraft interest calculated is wrong

2021-11-29 Thread GitBox
ptuomola merged pull request #1989: URL: https://github.com/apache/fineract/pull/1989 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] francisguchie commented on pull request #1984: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-29 Thread GitBox
francisguchie commented on pull request #1984: URL: https://github.com/apache/fineract/pull/1984#issuecomment-981928025 @fynmanoj This list is going to be hard coded i presume the number of counties in the world does not change that much and can be easily found here https://www.worldo

[GitHub] [fineract] fynmanoj commented on pull request #1984: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-11-29 Thread GitBox
fynmanoj commented on pull request #1984: URL: https://github.com/apache/fineract/pull/1984#issuecomment-982244722 @francisguchie isn't it a good idea to fetch the predefined data set from the platform itself?, especially when it is a mandatory field? That would help others to upgrade eff

[GitHub] [fineract] awasum commented on a change in pull request #1990: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

2021-11-30 Thread GitBox
awasum commented on a change in pull request #1990: URL: https://github.com/apache/fineract/pull/1990#discussion_r759049043 ## File path: README.md ## @@ -21,17 +21,17 @@ If you are interested in contributing to this project, but perhaps don't quite k Requirements ==

[GitHub] [fineract] ptuomola commented on a change in pull request #1990: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

2021-11-30 Thread GitBox
ptuomola commented on a change in pull request #1990: URL: https://github.com/apache/fineract/pull/1990#discussion_r759059231 ## File path: README.md ## @@ -21,17 +21,17 @@ If you are interested in contributing to this project, but perhaps don't quite k Requirements

[GitHub] [fineract] awasum commented on pull request #1990: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

2021-11-30 Thread GitBox
awasum commented on pull request #1990: URL: https://github.com/apache/fineract/pull/1990#issuecomment-982433917 I wonder if something like ./gradlew bootRun will still work? And will it be using the default MariaDB client now? How does this affect tomcat Deployments and reporting for cl

[GitHub] [fineract] awasum commented on pull request #1990: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

2021-11-30 Thread GitBox
awasum commented on pull request #1990: URL: https://github.com/apache/fineract/pull/1990#issuecomment-982437584 I will have to create time to test this today or tomorrow night... But don't let this stop you. @vidakovic, @IOhacker, @avikganguly01, @fynmanoj @rrpawar96 @francisguchie. This

[GitHub] [fineract] ptuomola commented on pull request #1990: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

2021-11-30 Thread GitBox
ptuomola commented on pull request #1990: URL: https://github.com/apache/fineract/pull/1990#issuecomment-982480788 > I wonder if something like ./gradlew bootRun will still work? > And will it be using the default MariaDB client now? Yes, bootRun will work as I've added the Mari

[GitHub] [fineract] vidakovic commented on pull request #1990: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

2021-11-30 Thread GitBox
vidakovic commented on pull request #1990: URL: https://github.com/apache/fineract/pull/1990#issuecomment-982532243 note: working on an easier approach for boot run... no need for mariadb parts after... On Tue, 30 Nov 2021, 11:06 Petri Tuomola, ***@***.***> wrote: > I wonder

[GitHub] [fineract] prakash24 opened a new pull request #1991: Monthly schedule february as first repay date

2021-12-01 Thread GitBox
prakash24 opened a new pull request #1991: URL: https://github.com/apache/fineract/pull/1991 ## Description The change aims to fix monthly schedule problem when the first repayment date is on end of February and subsequent repayment dates lies on 30th of every month.

[GitHub] [fineract] prakash24 closed pull request #1991: Monthly schedule february as first repay date

2021-12-02 Thread GitBox
prakash24 closed pull request #1991: URL: https://github.com/apache/fineract/pull/1991 -- 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. To unsubscribe, e-mail: commits-uns

[GitHub] [fineract] fynmanoj opened a new pull request #1992: FINERACT-1448 send-email-fix

2021-12-02 Thread GitBox
fynmanoj opened a new pull request #1992: URL: https://github.com/apache/fineract/pull/1992 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/p

[GitHub] [fineract] fynmanoj commented on pull request #1992: FINERACT-1448 send-email-fix

2021-12-02 Thread GitBox
fynmanoj commented on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-984676969 @ptuomola , Kindly review this. As you have added these lines, please let me know if you suspect any other impact for this change. -- This is an automated message from the

[GitHub] [fineract] fynmanoj closed pull request #1992: FINERACT-1448 send-email-fix

2021-12-02 Thread GitBox
fynmanoj closed pull request #1992: URL: https://github.com/apache/fineract/pull/1992 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] fynmanoj edited a comment on pull request #1992: FINERACT-1448 send-email-fix

2021-12-02 Thread GitBox
fynmanoj edited a comment on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-984676969 @ptuomola , Kindly review this. please let me know if you suspect any other impact for this change. -- This is an automated message from the Apache Git Service. To

[GitHub] [fineract] BLasan commented on pull request #1992: FINERACT-1448 send-email-fix

2021-12-02 Thread GitBox
BLasan commented on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-984859582 @ptuomola Could you please review this? With the `exclude` part, email sending was not getting executed (class not found error was throwing) -- This is an automated message fr

[GitHub] [fineract] ptuomola commented on pull request #1992: FINERACT-1448 send-email-fix

2021-12-02 Thread GitBox
ptuomola commented on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-985161158 @BLasan the build seems to be still failing due to duplicate classes... The reason why the exclusion is exactly to resolve this duplication of classes: otherwise we hav

[GitHub] [fineract] ptuomola edited a comment on pull request #1992: FINERACT-1448 send-email-fix

2021-12-02 Thread GitBox
ptuomola edited a comment on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-985161158 @BLasan the build seems to be still failing due to duplicate classes... The reason why the exclusion is exactly to resolve this duplication of classes: otherwise

[GitHub] [fineract] fynmanoj commented on pull request #1992: FINERACT-1448 send-email-fix

2021-12-02 Thread GitBox
fynmanoj commented on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-985257058 Hi @ptuomola , I have added the stack trace in this ticket https://issues.apache.org/jira/browse/FINERACT-1448 -- This is an automated message from the Apache Git Servic

[GitHub] [fineract] ptuomola opened a new pull request #1993: FINERACT-1448: Fix sending emails

2021-12-03 Thread GitBox
ptuomola opened a new pull request #1993: URL: https://github.com/apache/fineract/pull/1993 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/p

[GitHub] [fineract] ptuomola commented on pull request #1992: FINERACT-1448 send-email-fix

2021-12-03 Thread GitBox
ptuomola commented on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-985528490 I created a PR that hopefully fixes this - see #1993. Let's see if that works... -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [fineract] ptuomola commented on pull request #1993: FINERACT-1448: Fix sending emails

2021-12-03 Thread GitBox
ptuomola commented on pull request #1993: URL: https://github.com/apache/fineract/pull/1993#issuecomment-985532823 Hopefully this fixes the dependency issue that was stopping email sending from working. Now what would be really great is: some integration tests that confirm that bot

[GitHub] [fineract] fynmanoj commented on pull request #1992: FINERACT-1448 send-email-fix

2021-12-03 Thread GitBox
fynmanoj commented on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-985594991 @ptuomola , Thank you very much. This should work. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [fineract] fynmanoj closed pull request #1992: FINERACT-1448 send-email-fix

2021-12-03 Thread GitBox
fynmanoj closed pull request #1992: URL: https://github.com/apache/fineract/pull/1992 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] fynmanoj commented on pull request #1992: FINERACT-1448 send-email-fix

2021-12-03 Thread GitBox
fynmanoj commented on pull request #1992: URL: https://github.com/apache/fineract/pull/1992#issuecomment-985653156 Fixed by : https://github.com/apache/fineract/pull/1993 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [fineract] ptuomola merged pull request #1993: FINERACT-1448: Fix sending emails

2021-12-03 Thread GitBox
ptuomola merged pull request #1993: URL: https://github.com/apache/fineract/pull/1993 -- 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. To unsubscribe, e-mail: commits-unsu

[GitHub] [fineract] ptuomola opened a new pull request #1994: Implement integration tests for twofactor profile

2021-12-04 Thread GitBox
ptuomola opened a new pull request #1994: URL: https://github.com/apache/fineract/pull/1994 ## Description Describe the changes made and why they were made. Ignore if these details are present on the associated [Apache Fineract JIRA ticket](https://github.com/apache/fineract/p

[GitHub] [fineract] rrpawar96 commented on pull request #1984: FINERACT-1431:PassportDropDown-in-ClientIdentifier

2021-12-04 Thread GitBox
rrpawar96 commented on pull request #1984: URL: https://github.com/apache/fineract/pull/1984#issuecomment-986007523 @fynmanoj , please re-review this PR, now we are fetching the countries list from the database so it can be modified directly from the database. The documentTypeValue param

  1   2   3   4   5   6   7   8   9   10   >