Feature/38/get all users

[Kimoo193]

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 #

[Copilot]

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/controllers/user.controller.js:50

• The response structure in getAllUsers does not match the Swagger documentation in routes which expects properties 'success' and 'data'. Consider updating the response to { success: true, data: users } or modifying the documentation accordingly.

return res.status(200).json({ message: 'Users retrieved successfully', users, });

[Copilot]

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

[Copilot]

The response structure in getAllUsers does not match the Swagger documentation which expects 'success' and 'data' fields. Consider updating the response to align with the documented schema.

Suggested change

	message: 'Users retrieved successfully',
	users,
	success: true,
	data: users,

[Copilot]

The route documentation specifies security via bearerAuth, yet the 'authenticate' middleware is not applied. Consider adding the middleware to secure the endpoint.

Suggested change

	router.get('/api/users', getAllUsers);
	router.get('/api/users', authenticate, getAllUsers);