Skip to content

fineract-1665 and fineract-1666 combined fix#2441

Merged
fynmanoj merged 1 commit intoapache:developfrom
logoutdhaval:fineract-latest
Jul 23, 2022
Merged

fineract-1665 and fineract-1666 combined fix#2441
fynmanoj merged 1 commit intoapache:developfrom
logoutdhaval:fineract-latest

Conversation

@logoutdhaval
Copy link
Contributor

@logoutdhaval logoutdhaval commented Jul 21, 2022

Description

Describe the changes made and why they were made.
FINERACT-1665.
FINERACT-1666.
FINERACT-1655.
FINERACT-1647.
FINERACT-1648.

Checklist

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

  • Write the commit message as per https://github.com/apache/fineract/#pull-requests

  • Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

  • Create/update unit or integration tests for verifying the changes made.

  • Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

  • Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

  • Submission is not a "code dump". (Large changes can be made "in repository" via a branch. Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@logoutdhaval logoutdhaval force-pushed the fineract-latest branch 2 times, most recently from 7c77757 to f3b4be8 Compare July 22, 2022 18:11
@logoutdhaval logoutdhaval reopened this Jul 22, 2022
@fynmanoj fynmanoj closed this Jul 22, 2022
@fynmanoj fynmanoj reopened this Jul 22, 2022
@logoutdhaval logoutdhaval force-pushed the fineract-latest branch 2 times, most recently from 52884ea to 8151b44 Compare July 23, 2022 05:51
@fynmanoj fynmanoj merged commit d388a3a into apache:develop Jul 23, 2022
vidakovic pushed a commit that referenced this pull request Aug 3, 2022
Co-authored-by: Dhaval Maniyar <dhavalmaniyar@Dhavals-MacBook-Pro.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants