Closed singhcoder694 closed 8 months ago
@singhcoder694 is attempting to deploy a commit to the WebSorcery Team on Vercel.
A member of the Team first needs to authorize it.
The updates involve standardizing success messages in generator functions, streamlining email credentials in nodemailer configuration, refactoring API response handling for clarity, and adjusting HTTP status response logic in an API endpoint. These changes aim to simplify messaging, secure email transmission, and improve the clarity of API interactions.
File Path | Change Summary |
---|---|
.../generator/palindrome/palindromeGeneratorFunc.js .../generator/string/stringGeneratorFunc.js |
Unified success messages to "Values generated successfully!!" |
config/nodemailer.js |
Updated email credentials environment variables. |
lib/api.js |
Refactored response status handling to show success messages directly for successful statuses. |
pages/api/confirm.js |
Modified control flow to include explicit JSON status responses. |
🐇✨ To code, to build, a change we weave, Success messages, all in one sleeve. Emails secure, responses keen, A rabbit's touch, in code unseen. 🎉🥕
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
This PR is missing a Jira ticket reference in the title or description. Please add a Jira ticket reference to the title or description of this PR.
What I think is due to missing return statements mail was encountering 405 error otherwise it will not, you may try using your zoho id using by locating that in env file with same value as given in code @ank1raj
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
testcase-generator | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Dec 26, 2023 3:27pm |
@singhcoder694 No, it's still not fixed. let's sit together tomorrow and work on this and fix this asap.
Ok sir, I fixed it properly but if not we can meet to fix it.... but how can we meet? @ank1traj
🥷 Code experts: ank1traj
ank1traj has most 👩💻 activity in the files. ank1traj has most 🧠 knowledge in the files.
To learn more about /:\ gitStream - Visit our Docs
Corrected all error told by you everything is working perfectly fine. 405 error is also gone.... using gmail I have checked for many times now 405 is not there but couldn't try using Zoho since I don't have that account and it is not free to use.
Summary by CodeRabbit
New Features
Refactor
Bug Fixes