[GitHub] [fineract] vorburger opened a new pull request #692: add GitHub Action to autoclose stale pull requests

2020-01-13 Thread GitBox
vorburger opened a new pull request #692: add GitHub Action to autoclose stale pull requests URL: https://github.com/apache/fineract/pull/692 It will automatically add a comment after 30 days, and then another 30 days later (so after a total of 60 days of inactivity) automatically close

[GitHub] [fineract] vorburger commented on issue #684: more SpotBugs related / inspired code clean up (see FINERACT-702)

2020-01-13 Thread GitBox
vorburger commented on issue #684: more SpotBugs related / inspired code clean up (see FINERACT-702) URL: https://github.com/apache/fineract/pull/684#issuecomment-573551071 @awasum this now passed the build (because it's Monday), and is finished and ready from my side. Will you

[fineract] annotated tag release-1.4.0 created (now 5d9773d)

2020-01-13 Thread nazeer1100126
This is an automated email from the ASF dual-hosted git repository. nazeer1100126 pushed a change to annotated tag release-1.4.0 in repository https://gitbox.apache.org/repos/asf/fineract.git. at 5d9773d (tag) tagging 7420545d8f9c35276954def94a58800b29bdde93 (commit) replaces 1.3.0

[GitHub] [fineract] vorburger merged pull request #693: fix build.gradle for RAT to ignore .github/workflows/

2020-01-13 Thread GitBox
vorburger merged pull request #693: fix build.gradle for RAT to ignore .github/workflows/ URL: https://github.com/apache/fineract/pull/693 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [fineract] vorburger commented on issue #692: add GitHub Action to autoclose stale pull requests

2020-01-13 Thread GitBox
vorburger commented on issue #692: add GitHub Action to autoclose stale pull requests URL: https://github.com/apache/fineract/pull/692#issuecomment-573563316 @awasum and anyone else reading this, how about adding this? This

[fineract] branch develop updated (7420545 -> 0379ba7)

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/fineract.git. from 7420545 Merge pull request #690 from vorburger/FINERACT-773_fix-docker-compose add 0379ba7 fix

[fineract] branch vorburger-stale-action updated (a8c0e97 -> 286277d)

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a change to branch vorburger-stale-action in repository https://gitbox.apache.org/repos/asf/fineract.git. discard a8c0e97 add GitHub Action to autoclose stale pull requests add 0379ba7 fix build.gradle

[fineract] branch vorburger-slash-rebase-pr-comment-action updated (ad56f0d -> aa310aa)

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a change to branch vorburger-slash-rebase-pr-comment-action in repository https://gitbox.apache.org/repos/asf/fineract.git. discard ad56f0d add /rebase comment support for Pull Requests add aa310aa add

[fineract] 01/01: add GitHub Action to autoclose stale pull requests

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a commit to branch vorburger-stale-action in repository https://gitbox.apache.org/repos/asf/fineract.git commit a8c0e97217616175a7a8f54ae598701115d99a46 Author: Michael Vorburger ⛑️ AuthorDate: Mon Jan 13

[fineract] branch vorburger-stale-action created (now a8c0e97)

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a change to branch vorburger-stale-action in repository https://gitbox.apache.org/repos/asf/fineract.git. at a8c0e97 add GitHub Action to autoclose stale pull requests This branch includes the following

[fineract] branch 1.4.0 created (now 7420545)

2020-01-13 Thread nazeer1100126
This is an automated email from the ASF dual-hosted git repository. nazeer1100126 pushed a change to branch 1.4.0 in repository https://gitbox.apache.org/repos/asf/fineract.git. at 7420545 Merge pull request #690 from vorburger/FINERACT-773_fix-docker-compose No new revisions were

[GitHub] [fineract] vorburger commented on issue #691: add /rebase comment support for Pull Requests

2020-01-13 Thread GitBox
vorburger commented on issue #691: add /rebase comment support for Pull Requests URL: https://github.com/apache/fineract/pull/691#issuecomment-573554719 #693 will fix the build failure seen here This is an automated message

[fineract] branch vorburger-slash-rebase-pr-comment-action updated (aa310aa -> 6c0755a)

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a change to branch vorburger-slash-rebase-pr-comment-action in repository https://gitbox.apache.org/repos/asf/fineract.git. discard aa310aa add /rebase comment support for Pull Requests add 0379ba7 fix

[GitHub] [fineract] vorburger commented on issue #692: add GitHub Action to autoclose stale pull requests

2020-01-13 Thread GitBox
vorburger commented on issue #692: add GitHub Action to autoclose stale pull requests URL: https://github.com/apache/fineract/pull/692#issuecomment-573791466 The test build failure seen here seems arbitrary, I've created [FINERACT-828](https://issues.apache.org/jira/browse/FINERACT-828),

[GitHub] [fineract] vorburger closed pull request #692: add GitHub Action to autoclose stale pull requests

2020-01-13 Thread GitBox
vorburger closed pull request #692: add GitHub Action to autoclose stale pull requests URL: https://github.com/apache/fineract/pull/692 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [fineract] vorburger opened a new pull request #692: add GitHub Action to autoclose stale pull requests

2020-01-13 Thread GitBox
vorburger opened a new pull request #692: add GitHub Action to autoclose stale pull requests URL: https://github.com/apache/fineract/pull/692 It will automatically add a comment after 30 days, and then another 30 days later (so after a total of 60 days of inactivity) automatically close

[GitHub] [fineract] vorburger opened a new pull request #685: clean up GLAccountBuilder, following SpotBugs (FINERACT-702)

2020-01-13 Thread GitBox
vorburger opened a new pull request #685: clean up GLAccountBuilder, following SpotBugs (FINERACT-702) URL: https://github.com/apache/fineract/pull/685 This is an automated message from the Apache Git Service. To respond

[GitHub] [fineract] vorburger closed pull request #684: more SpotBugs related / inspired code clean up (see FINERACT-702)

2020-01-13 Thread GitBox
vorburger closed pull request #684: more SpotBugs related / inspired code clean up (see FINERACT-702) URL: https://github.com/apache/fineract/pull/684 This is an automated message from the Apache Git Service. To respond

[GitHub] [fineract] vorburger closed pull request #685: clean up GLAccountBuilder, following SpotBugs (FINERACT-702)

2020-01-13 Thread GitBox
vorburger closed pull request #685: clean up GLAccountBuilder, following SpotBugs (FINERACT-702) URL: https://github.com/apache/fineract/pull/685 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [fineract] vorburger opened a new pull request #684: more SpotBugs related / inspired code clean up (see FINERACT-702)

2020-01-13 Thread GitBox
vorburger opened a new pull request #684: more SpotBugs related / inspired code clean up (see FINERACT-702) URL: https://github.com/apache/fineract/pull/684 based upon code review of changes made by @awasum in previous commit: - fix clear contains() GC_UNRELATED_TYPES bug in Loan

[GitHub] [fineract] vorburger commented on issue #664: FINERACT-802-restrict-client-to-have-single-self-service-user

2020-01-13 Thread GitBox
vorburger commented on issue #664: FINERACT-802-restrict-client-to-have-single-self-service-user URL: https://github.com/apache/fineract/pull/664#issuecomment-573794166 jamesidw@ yes sorry you are absolutely correct of course... ;-) aimanoj@ will you rebase and resolve conflicts for

[GitHub] [fineract] vorburger commented on issue #592: FINERACT-614: Rates Module

2020-01-13 Thread GitBox
vorburger commented on issue #592: FINERACT-614: Rates Module URL: https://github.com/apache/fineract/pull/592#issuecomment-573793341 > Ok, I will rebase the branch with dev. No problem @angelboxes were you still planning to do that? Just asking. If not, would you like to close this

[GitHub] [fineract] jamesidw commented on issue #664: FINERACT-802-restrict-client-to-have-single-self-service-user

2020-01-13 Thread GitBox
jamesidw commented on issue #664: FINERACT-802-restrict-client-to-have-single-self-service-user URL: https://github.com/apache/fineract/pull/664#issuecomment-573663057 @aimanoj I believe that should have been directed at you (since this PR comes from your repository) @vorburger Don't

[GitHub] [fineract] awasum commented on issue #692: add GitHub Action to autoclose stale pull requests

2020-01-13 Thread GitBox
awasum commented on issue #692: add GitHub Action to autoclose stale pull requests URL: https://github.com/apache/fineract/pull/692#issuecomment-573834932 > @awasum and anyone else reading this, how about adding this? This makes sense to have a 60 day auto close rule. I have no

[fineract] branch Fineract-614 updated (bf0393b -> 565999a)

2020-01-13 Thread angelboxes
This is an automated email from the ASF dual-hosted git repository. angelboxes pushed a change to branch Fineract-614 in repository https://gitbox.apache.org/repos/asf/fineract.git. from bf0393b Fixing some code to be compatible with Spring update add 565999a Fineract-614: Renaming

[fineract] branch Fineract-614 updated (867e7e1 -> bf0393b)

2020-01-13 Thread angelboxes
This is an automated email from the ASF dual-hosted git repository. angelboxes pushed a change to branch Fineract-614 in repository https://gitbox.apache.org/repos/asf/fineract.git. discard 867e7e1 Merge branch 'develop' into Fineract-614 discard c66c55f FINERACT-737: Updated Api Docs to

[GitHub] [fineract] awasum commented on issue #688: clean up all whitespace issues in Java files (FINERACT-821)

2020-01-13 Thread GitBox
awasum commented on issue #688: clean up all whitespace issues in Java files (FINERACT-821) URL: https://github.com/apache/fineract/pull/688#issuecomment-574042839 > @awasum I'll fix the Merge Conflicts on this one **AFTER** my other work in #684 and #685 has been reviewed and merged. (I

[GitHub] [fineract] vorburger commented on issue #691: add /rebase comment support for Pull Requests

2020-01-13 Thread GitBox
vorburger commented on issue #691: add /rebase comment support for Pull Requests URL: https://github.com/apache/fineract/pull/691#issuecomment-574045294 self merging this one based on @awasum comment in #692 ("no experience with Github Actions. Merge when you feel comfortable with this.")

[GitHub] [fineract] vorburger merged pull request #691: add /rebase comment support for Pull Requests

2020-01-13 Thread GitBox
vorburger merged pull request #691: add /rebase comment support for Pull Requests URL: https://github.com/apache/fineract/pull/691 This is an automated message from the Apache Git Service. To respond to the message, please

[fineract] branch develop updated: add /rebase comment support for Pull Requests

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/fineract.git The following commit(s) were added to refs/heads/develop by this push: new 653b0b7 add /rebase comment support for

[GitHub] [fineract] vorburger commented on a change in pull request #629: Swagger documentation gsoc2019

2020-01-13 Thread GitBox
vorburger commented on a change in pull request #629: Swagger documentation gsoc2019 URL: https://github.com/apache/fineract/pull/629#discussion_r366189471 ## File path: fineract-provider/src/main/java/org/apache/fineract/portfolio/floatingrates/api/FloatingRatesApiResource.java

[GitHub] [fineract] awasum commented on issue #629: Swagger documentation gsoc2019

2020-01-13 Thread GitBox
awasum commented on issue #629: Swagger documentation gsoc2019 URL: https://github.com/apache/fineract/pull/629#issuecomment-574038079 @Sanyam96 , @vorburger , Seems conflicts are now fixed by @kangbreder and Travis is passing. Merge this when yo feel comfortable... instead create more

[GitHub] [fineract] vorburger merged pull request #692: add GitHub Action to autoclose stale pull requests

2020-01-13 Thread GitBox
vorburger merged pull request #692: add GitHub Action to autoclose stale pull requests URL: https://github.com/apache/fineract/pull/692 This is an automated message from the Apache Git Service. To respond to the message,

[fineract] branch develop updated (0379ba7 -> 54b7ddf)

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/fineract.git. from 0379ba7 fix build.gradle for RAT to ignore .github/workflows/ add 54b7ddf add GitHub Action to

[GitHub] [fineract] vorburger commented on issue #688: clean up all whitespace issues in Java files (FINERACT-821)

2020-01-13 Thread GitBox
vorburger commented on issue #688: clean up all whitespace issues in Java files (FINERACT-821) URL: https://github.com/apache/fineract/pull/688#issuecomment-574040714 @awasum I'll fix the Merge Conflicts on this one **AFTER** my other work in #684 and #685 has been reviewed and merged. (I

[GitHub] [fineract] vorburger commented on issue #688: clean up all whitespace issues in Java files (FINERACT-821)

2020-01-13 Thread GitBox
vorburger commented on issue #688: clean up all whitespace issues in Java files (FINERACT-821) URL: https://github.com/apache/fineract/pull/688#issuecomment-574043410 > Makes sense. Go for it. but I'm waiting for you to review and hopefully merge #684 and #685 ... :smiley:

[GitHub] [fineract] awasum commented on issue #691: add /rebase comment support for Pull Requests

2020-01-13 Thread GitBox
awasum commented on issue #691: add /rebase comment support for Pull Requests URL: https://github.com/apache/fineract/pull/691#issuecomment-574043551 @vorburger Do we need this documented somewhere on the readme. It can be done after this has been merged if you want. Looks like just

[GitHub] [fineract] vorburger commented on a change in pull request #629: Swagger documentation gsoc2019

2020-01-13 Thread GitBox
vorburger commented on a change in pull request #629: Swagger documentation gsoc2019 URL: https://github.com/apache/fineract/pull/629#discussion_r366188664 ## File path: fineract-provider/build.gradle ## @@ -59,6 +60,8 @@ apply plugin: 'eclipse' apply plugin: 'idea'

[GitHub] [fineract] vorburger commented on issue #629: Swagger documentation gsoc2019

2020-01-13 Thread GitBox
vorburger commented on issue #629: Swagger documentation gsoc2019 URL: https://github.com/apache/fineract/pull/629#issuecomment-574047849 /rebase This is an automated message from the Apache Git Service. To respond to the

[GitHub] [fineract] awasum commented on issue #592: FINERACT-614: Rates Module

2020-01-13 Thread GitBox
awasum commented on issue #592: FINERACT-614: Rates Module URL: https://github.com/apache/fineract/pull/592#issuecomment-574036658 @angelboxes , we now have spotbugs enabled for Fineract. So check to make sure your changes do not fail spotbug rules. See #670 #673 Fineract has also

[fineract] branch develop updated (54b7ddf -> efb0a55)

2020-01-13 Thread vorburger
This is an automated email from the ASF dual-hosted git repository. vorburger pushed a change to branch develop in repository https://gitbox.apache.org/repos/asf/fineract.git. from 54b7ddf add GitHub Action to autoclose stale pull requests add efb0a55 FINERACT-824: Removed the second

[GitHub] [fineract] vorburger merged pull request #683: FINERACT-824: NullPointerException is thrown when submitting a loan account application with an existing externalId

2020-01-13 Thread GitBox
vorburger merged pull request #683: FINERACT-824: NullPointerException is thrown when submitting a loan account application with an existing externalId URL: https://github.com/apache/fineract/pull/683 This is an automated

[GitHub] [fineract] awasum merged pull request #685: clean up GLAccountBuilder, following SpotBugs (FINERACT-702)

2020-01-13 Thread GitBox
awasum merged pull request #685: clean up GLAccountBuilder, following SpotBugs (FINERACT-702) URL: https://github.com/apache/fineract/pull/685 This is an automated message from the Apache Git Service. To respond to the

[fineract] branch develop updated: clean up GLAccountBuilder, following SpotBugs (FINERACT-702)

2020-01-13 Thread awasum
This is an automated email from the ASF dual-hosted git repository. awasum pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/fineract.git The following commit(s) were added to refs/heads/develop by this push: new a5c60b0 clean up GLAccountBuilder,

[GitHub] [fineract] awasum merged pull request #684: more SpotBugs related / inspired code clean up (see FINERACT-702)

2020-01-13 Thread GitBox
awasum merged pull request #684: more SpotBugs related / inspired code clean up (see FINERACT-702) URL: https://github.com/apache/fineract/pull/684 This is an automated message from the Apache Git Service. To respond to

[GitHub] [fineract] awasum commented on issue #688: clean up all whitespace issues in Java files (FINERACT-821)

2020-01-13 Thread GitBox
awasum commented on issue #688: clean up all whitespace issues in Java files (FINERACT-821) URL: https://github.com/apache/fineract/pull/688#issuecomment-574044939 > > Makes sense. Go for it. > > but I'm waiting for you to review and hopefully merge #684 and #685 ... I think

[GitHub] [fineract] awasum commented on issue #685: clean up GLAccountBuilder, following SpotBugs (FINERACT-702)

2020-01-13 Thread GitBox
awasum commented on issue #685: clean up GLAccountBuilder, following SpotBugs (FINERACT-702) URL: https://github.com/apache/fineract/pull/685#issuecomment-574044586 This looks good merging now. This is an automated message