FINERACT 263 : "Add Notes" for Centers permission is missing

[Monica-CodingWorld]

Description

This PR fixes missing permission handling for Center Notes in the Notes API.

Previously, when accessing notes for a CENTER resource, the request fell into the default case, resulting in a NoteResourceNotSupportedException. This allowed center notes to be accessed without proper, explicit permission mapping.

changes made:

• Added CENTER enum value to NoteType
• Added centerId field and withCenterId() method to CommandWrapperBuilder
• Added CENTER case in NotesApiResource.getResourceDetails()
• Enable separate permissions for Read, Create, Update, Delete center notes
• Follows same permission pattern as existing GROUP notes

Jira issue:- https://issues.apache.org/jira/projects/FINERACT/issues/FINERACT-263

Checklist

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

• Write the commit message as per our guidelines
• 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 our 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
• This PR must not be a "code dump". Large changes can be made in a branch, with assistance. Ask for help on the developer mailing list.

Your assigned reviewer(s) will follow our guidelines for code reviews.

[Monica-CodingWorld]

Hi reviewers,
This change updates the default handling in NotesApiResource#getResourceDetails to throw NoteResourceNotSupportedException instead of falling back to INVALIDNOTE, as discussed in FINERACT-263.Please let me know if there are any other files or cases you’d recommend updating as part of this fix.
I’d be happy to incorporate your feedback.
Thanks!

[adamsaghy]

PR content has nothing to do with PR title...

[Monica-CodingWorld]

PR content has nothing to do with PR title...

Really sorry all changes did not got commited. I'll update it shortly.

[IOhacker]

@Monica-CodingWorld squash and commit your changes, make sure that only 1 commit is in the PR. Also you have to test it in your local dev env.

[Monica-CodingWorld]

@Monica-CodingWorld squash and commit your changes, make sure that only 1 commit is in the PR. Also you have to test it in your local dev env.

@IOhacker Thank you for the guidance, i made all the changes.
The PR is now ready with a clean commit history. Could you please review when you have a moment?

Best Regards,
Monica

[IOhacker]

@Monica-CodingWorld could you please check the output https://github.com/apache/fineract/actions/runs/21178884898/job/61230181309?pr=5346

[Monica-CodingWorld]

@Monica-CodingWorld could you please check the output https://github.com/apache/fineract/actions/runs/21178884898/job/61230181309?pr=5346

@IOhacker Thanks for the review. I've looked into the CI build failure. The error occurs because the addition of the centerId parameter to the CommandWrapper constructors requires updates in a few more places that instantiate the class. I have now identified and fixed these remaining instances (added null for the centerId parameter where appropriate) and have pushed the corrections. The build should pass now.

[IOhacker]

:) always remember to squash and commit. Please help us with that.