feat(api): Added feedback form module

[rayaanoidPrime]

Description

• Created a module named feedback
• Added a controller and a service class in it.
• Added a function called registerFeedback(feedback: string) in the service class
• Added a POST mapping in the feedback controller. The controller is mapped under /api/feedback
• Marked the controller function as @public()
• Validating that the feedback isn't null in the service class. Emailing admin@keyshade.xyz the feedback message using MailService
• Added relevant tests for this in feedback.e2e.spec.ts

Fixes #205

Dependencies

None

Future Improvements

None

Mentions

None

Screenshots of relevant screens

None

Developer's checklist

• My PR follows the style guidelines of this project
• I have performed a self-check on my work

If changes are made in the code:

• I have followed the coding guidelines
• My changes in code generate no new warnings
• [] My changes are breaking another fix/feature of the project
• I have added test cases to show that my feature works
• I have added relevant screenshots in my PR
• There are no UI/UX issues

Documentation Update

• This PR requires an update to the documentation at docs.keyshade.xyz
• I have made the necessary updates to the documentation, or no documentation changes are required.

[codiumai-pr-agent-free]

Failed to generate code suggestions for PR

[rajdip-b]

/review

[codiumai-pr-agent-free]

PR Review 🔍

⏱️ Estimated effort to review [1-5]	2, because the PR is well-structured with clear separation of concerns, includes tests, and follows the project's coding guidelines. However, the changes are localized to a new module, making it easier to review.
🧪 Relevant tests	Yes
⚡ Possible issues	Possible Bug: The feedback registration does not handle cases where the feedback string might be only whitespace. This could lead to unintended behavior where seemingly empty feedback is accepted.
🔒 Security concerns	No

Code feedback:

---

relevant file	apps/api/src/feedback/service/feedback.service.ts
suggestion	Consider trimming the feedback string before checking if it is empty to handle cases where the feedback consists only of whitespace. This can prevent accepting empty-like strings which are technically not null. [important]
relevant line	if (!feedback) {

---

relevant file	apps/api/src/feedback/controller/feedback.controller.ts
suggestion	Add HTTP status code 201 for successful feedback registration to align with RESTful practices for resource creation. Modify the registerFeedback method to return { statusCode: 201 }. [medium]
relevant line	async registerFeedback(@Body('feedback') feedback: string): Promise {

---

[rajdip-b]

Can you just get this resoved? https://sonarcloud.io/project/issues?resolved=false&sinceLeakPeriod=true&pullRequest=210&id=keyshade-xyz_keyshade&open=AY9tnJtdrNHifkb_0SR1

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[rajdip-b]

🎉 This PR is included in version 1.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀