Skip to content

FINERACT-1089: Fixing 'Operation not allowed for a result set of type"#1980

Merged
vorburger merged 1 commit into
apache:developfrom
ptuomola:FINERACT-1089a
Dec 23, 2021
Merged

FINERACT-1089: Fixing 'Operation not allowed for a result set of type"#1980
vorburger merged 1 commit into
apache:developfrom
ptuomola:FINERACT-1089a

Conversation

@ptuomola
Copy link
Copy Markdown
Contributor

@ptuomola ptuomola commented Nov 24, 2021

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!

  • 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/api-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.

@ptuomola ptuomola closed this Nov 25, 2021
@ptuomola ptuomola reopened this Nov 25, 2021
@ptuomola
Copy link
Copy Markdown
Contributor Author

@vorburger this should fix one of the error messages you were seeing in the logs of fineract.dev. Btw it might be that someone has changed the password for the mifos account again? If yes we should really block that..

@vorburger vorburger merged commit 7b4fcb9 into apache:develop Dec 23, 2021
@vorburger
Copy link
Copy Markdown
Member

@vorburger this should fix one of the error messages you were seeing in the logs of fineract.dev. Btw it might be that someone has changed the password for the mifos account again? If yes we should really block that..

Cool! Ah yes I really should fix that ASAP. Could you keep bugging me about it? 😃 Hopefully I can get to it over the next week or two. I did kind of forget that, and am currently having much fun elsewhere... (There is that certain other little Apache project a few people lately heard a few things about, where I feel strangely compelled to make it a better place; FYI apache/logging-log4j2@release-2.x...vorburger:release-2.x_split-API and some related activities - do give me a shout by DM if you have any interest in putting our heads together about that?)

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