Solution #806
Solution #806
Conversation
There was a problem hiding this comment.
Great job on implementing the solution! 🎉 The core functionality is working well, and the code meets the task requirements. There are just a couple of minor issues to address: correcting the import path for sendJSONResponse and ensuring the convertErrorMessage format is URL-friendly. These are small tweaks and don't impact the overall functionality. Keep up the excellent work, and consider these improvements for future projects! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| class ErrorCase { | ||
| caseTypes = ['SNAKE', 'KEBAB', 'CAMEL', 'PASCAL', 'UPPER']; | ||
|
|
||
| convertErrorMessage = '/<TEXT_TO_CONVERT>?toCase=<CASE_NAME>'; |
There was a problem hiding this comment.
The convertErrorMessage string uses a placeholder format that might not be correctly interpreted if used directly in a URL. Consider using a more explicit format or providing an example of how the URL should be structured.
|
|
||
| const { convertToCase } = require('./convertToCase/convertToCase'); | ||
| const { errorCase } = require('./Error'); | ||
| const { sendJSONResponse } = require('./senResponse'); |
There was a problem hiding this comment.
There is a typo in the import path for sendJSONResponse. It should be ./sendResponse instead of ./senResponse to match the file name.
Solution