feature/delete-organization-pic

[mdawoud27]

PR Checklist (required)

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines.

• Tests for the changes have been added (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation changes
• Other... Please describe:

Related Issue

Closes #70

[Copilot]

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/routes/organization.routes.js:60

• [nitpick] Consider using a more RESTful route path by omitting the '/delete' suffix, since the HTTP DELETE method implies the delete action. For example, use '/api/organization/:organizationId/logo'.

'/api/organization/:organizationId/logo/delete',

[Copilot]

The extraction of the publicId using string splitting may fail if the imageUrl contains query parameters. Consider using a robust URL parsing method (e.g., the URL constructor) to reliably extract the publicId.

Suggested change

	const publicId = imageUrl.split('/').pop().split('.')[0];
	const url = new URL(imageUrl);
	const publicId = url.pathname.split('/').pop().split('.')[0];