-
-
Notifications
You must be signed in to change notification settings - Fork 103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add feat/ContactUs-form branch #381
Changes from 1 commit
56f7052
a03eecc
cfc2fa8
cc084c0
6a933b9
012d5ae
a760726
689029b
58fcfba
90e0fa0
1f8f7e3
c9bf8af
429c94e
c472329
ff1422f
8cebaec
bb5409f
6426ecd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,73 @@ | ||||||||||||||||||||||||||||||||||||||||
const { z } = require("zod"); | ||||||||||||||||||||||||||||||||||||||||
const nodemailer = require("nodemailer"); | ||||||||||||||||||||||||||||||||||||||||
require("dotenv").config(); | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
// data require form .env file : EMAIL_USER, EMAIL_PASS | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
// Define the Zod schema for contact form validation | ||||||||||||||||||||||||||||||||||||||||
const contactSchema = z.object({ | ||||||||||||||||||||||||||||||||||||||||
mail: z.string().email(), | ||||||||||||||||||||||||||||||||||||||||
subject: z.string().min(5, "Subject must be at least 5 characters long."), | ||||||||||||||||||||||||||||||||||||||||
message: z.string().min(5, "Message must be at least 5 characters long."), | ||||||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
const info = { messageId: "23DCS141" }; | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
const createContactUs = async (req, res) => { | ||||||||||||||||||||||||||||||||||||||||
const validation = contactSchema.safeParse(req.body); | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
if (!validation.success) { | ||||||||||||||||||||||||||||||||||||||||
console.log("Error at validation"); | ||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Use Using Apply this diff to update the logging statements: // In validation failure
- console.log("Error at validation");
+ console.error("Error at validation");
// In sendMail error handling
- return console.log("Error occurred: " + error.message);
+ console.error("Error occurred: " + error.message);
// In catch block
- console.log(`Error at transport: ${err}`);
+ console.error(`Error at transport: ${err}`); Also applies to: 54-54, 64-64 |
||||||||||||||||||||||||||||||||||||||||
return res.status(400).json({ | ||||||||||||||||||||||||||||||||||||||||
status: "error", | ||||||||||||||||||||||||||||||||||||||||
errors: "contactSchema is not validate", | ||||||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Improve error handling and messages on validation failure. The error message "contactSchema is not validate" is grammatically incorrect. Additionally, consider providing detailed validation errors to the client for better clarity. Apply this diff to fix the error message and include validation details: console.log("Error at validation");
return res.status(400).json({
status: "error",
- errors: "contactSchema is not validate",
+ errors: validation.error.errors,
}); This change returns the specific validation errors provided by Zod, helping the client understand what went wrong. 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
const { mail, subject, message } = req.body; | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
try { | ||||||||||||||||||||||||||||||||||||||||
const transporter = nodemailer.createTransport({ | ||||||||||||||||||||||||||||||||||||||||
service: "gmail", | ||||||||||||||||||||||||||||||||||||||||
host: "smtp.gmail.com", | ||||||||||||||||||||||||||||||||||||||||
port: 587, | ||||||||||||||||||||||||||||||||||||||||
secure: false, | ||||||||||||||||||||||||||||||||||||||||
auth: { | ||||||||||||||||||||||||||||||||||||||||
user: process.env.EMAIL_USER, | ||||||||||||||||||||||||||||||||||||||||
pass: process.env.EMAIL_PASS, | ||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||
tls: { | ||||||||||||||||||||||||||||||||||||||||
rejectUnauthorized: false, // Disable strict SSL verification | ||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Enable SSL verification in transporter configuration. Disabling SSL verification by setting Apply this diff to enable SSL verification: tls: {
- rejectUnauthorized: false, // Disable strict SSL verification
+ // Removed to enable strict SSL verification
}, If the 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
const mailOptions = { | ||||||||||||||||||||||||||||||||||||||||
from: mail, | ||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Avoid setting 'from' to a user-provided email to prevent spoofing. Setting the Apply this diff to fix the issue: const mailOptions = {
- from: mail,
+ from: process.env.EMAIL_USER,
+ replyTo: mail,
to: process.env.EMAIL_USER,
subject: subject,
text: message,
}; This ensures that the email is sent from a trusted address while allowing you to reply to the user's email. 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||
to: process.env.EMAIL_USER, | ||||||||||||||||||||||||||||||||||||||||
subject: subject, | ||||||||||||||||||||||||||||||||||||||||
text: message, | ||||||||||||||||||||||||||||||||||||||||
}; | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
// Send mail with defined transport object | ||||||||||||||||||||||||||||||||||||||||
transporter.sendMail(mailOptions, (error, mailOptions) => { | ||||||||||||||||||||||||||||||||||||||||
if (error) { | ||||||||||||||||||||||||||||||||||||||||
return console.log("Error occurred: " + error.message); | ||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Handle errors in Currently, if Apply this diff to handle errors and send appropriate responses: // Send mail with defined transport object
transporter.sendMail(mailOptions, (error, info) => {
if (error) {
- return console.log("Error occurred: " + error.message);
+ console.error("Error occurred: " + error.message);
+ return res.status(500).json({
+ status: "error",
+ message: "There was an error sending your message. Please try again later.",
+ });
}
-
+ res.status(200).json({
+ status: "success",
+ message: "Your contact request has been successfully received.",
+ });
});
-
-res.status(200).json({
- status: "success",
- message: "Your contact request has been successfully received.",
-}); This change moves the success response inside the callback after the email is sent successfully and sends an error response if there's a failure. 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
res.status(200).json({ | ||||||||||||||||||||||||||||||||||||||||
status: "success", | ||||||||||||||||||||||||||||||||||||||||
message: "Your contact request has been successfully received.", | ||||||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||||||
} catch (err) { | ||||||||||||||||||||||||||||||||||||||||
console.log(`Error at transport: ${err}`); | ||||||||||||||||||||||||||||||||||||||||
res.status(500).json({ | ||||||||||||||||||||||||||||||||||||||||
status: "error", | ||||||||||||||||||||||||||||||||||||||||
message: | ||||||||||||||||||||||||||||||||||||||||
"There was an error sending your message. Please try again later.", | ||||||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||
}; | ||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||
module.exports = { createContactUs }; |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,7 +3,7 @@ require("dotenv").config(); | |
const cors = require("cors"); | ||
const mongoose = require("mongoose"); | ||
const logger = require("./config/logger"); | ||
const errorMiddleware = require("./middlewares/errorMiddleware"); // Corrected typo | ||
const errorMiddleware = require("../backend/middlewares/errrorMiddleware"); // Corrected typo | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Fix typo in error middleware filename and import path The error middleware file is incorrectly named with an extra 'r', which can lead to confusion and potential issues.
🔗 Analysis chainFix typo in error middleware import path The import statement for the error middleware contains a typo in the file name. While attempting to correct a previous issue, a new typo has been introduced. Please apply the following change: - const errorMiddleware = require("../backend/middlewares/errrorMiddleware"); // Corrected typo
+ const errorMiddleware = require("../backend/middlewares/errorMiddleware"); // Corrected typo Also, please verify that this is indeed the correct path to the error middleware file. The change from To verify the correct path of the error middleware file, run the following command: This will help confirm the correct location of the file and ensure that the import statement is accurate. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Locate the error middleware file and verify its path
# Test: Search for the error middleware file
fd errorMiddleware.js
Length of output: 23 Script: #!/bin/bash
# Description: Locate the incorrectly named error middleware file to verify the typo
# Test: Search for the incorrectly spelled error middleware file
fd errrorMiddleware.js
Length of output: 64 |
||
const passport = require("passport"); | ||
const { handleGoogleOAuth } = require("./controller/googleOAuth.controller"); | ||
const app = express(); | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
const express = require("express"); | ||
const router = express.Router(); | ||
const { createContactUs } = require("../controller/contact.controller"); // Correct controller path | ||
|
||
router.post("/contactus", createContactUs); | ||
|
||
module.exports = router; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,57 +1,45 @@ | ||
const express = require("express"); | ||
const logger = require("../config/logger"); // Import your Winston logger | ||
const logger = require("../config/logger"); // Import Winston logger | ||
require("dotenv").config(); | ||
|
||
const config = { | ||
JWT_SECRET: process.env.JWT_SECRET, | ||
GOOGLE_CLIENT_ID: process.env.GOOGLE_CLIENT_ID, | ||
GOOGLE_CLIENT_SECRET: process.env.GOOGLE_CLIENT_SECRET, | ||
}; | ||
|
||
const router = express.Router(); | ||
|
||
let feedbackRouter; | ||
|
||
try { | ||
feedbackRouter = require("./feedbackRouter"); | ||
} catch (error) { | ||
logger.error("Error loading feedbackRouter:", error); // Log the error with Winston | ||
feedbackRouter = (req, res) => { | ||
res | ||
.status(500) | ||
.json({ error: "Feedback functionality is currently unavailable" }); | ||
}; | ||
} | ||
// Utility function to safely load modules and handle errors | ||
const safeRequire = (modulePath, fallbackMessage) => { | ||
try { | ||
return require(modulePath); | ||
} catch (error) { | ||
logger.error(`Error loading ${modulePath}:`, error); | ||
return (req, res) => { | ||
res.status(500).json({ error: fallbackMessage }); | ||
}; | ||
} | ||
}; | ||
|
||
let eventRouter; | ||
try { | ||
eventRouter = require("./eventRouter"); | ||
} catch (error) { | ||
logger.error("Error loading eventRouter:", error); // Log the error with Winston | ||
eventRouter = (req, res) => { | ||
res | ||
.status(500) | ||
.json({ error: "Event functionality is currently unavailable" }); | ||
}; | ||
} | ||
// Safely load routers with error handling | ||
const feedbackRouter = safeRequire("./feedbackRouter", "Feedback functionality is currently unavailable"); | ||
const contactUsRouter = safeRequire("./contactUsRouter", "Contact Us functionality is currently unavailable"); | ||
const eventRouter = safeRequire("./eventRouter", "Event functionality is currently unavailable"); | ||
|
||
router.get("/", (req, res) => { | ||
return res.json({ | ||
message: "Welcome to the restaurant API!", | ||
version: "1.0.0", | ||
endpoints: { | ||
Reservation: "/reservation", | ||
Feedback: "/feedback", // Added feedback endpoint documentation | ||
Feedback: "/feedback", | ||
}, | ||
documentation: "https://api-docs-url.com", | ||
}); | ||
}); | ||
|
||
router.use("/event", eventRouter); | ||
router.use("/admin", require("./adminRouter")); | ||
router.use("/admin", safeRequire("./adminRouter", "Admin functionality is currently unavailable")); | ||
router.use("/feedback", feedbackRouter); | ||
router.use("/user", require("./customerRouter")); | ||
router.use("/reservation", require("./reservationRouter")); | ||
router.use("/newsletter", require("./newsletterRoute")); | ||
router.use("/forgot", require("./forgotRouter")); | ||
router.use("/user", safeRequire("./customerRouter", "User functionality is currently unavailable")); | ||
router.use("/reservation", safeRequire("./reservationRouter", "Reservation functionality is currently unavailable")); | ||
router.use("/newsletter", safeRequire("./newsletterRoute", "Newsletter functionality is currently unavailable")); | ||
router.use("/forgot", safeRequire("./forgotRouter", "Forgot password functionality is currently unavailable")); | ||
router.use("/contact", contactUsRouter); | ||
|
||
module.exports = router; |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,165 @@ | ||||||||||||||||||||||||||||||||||||
/* eslint-disable prettier/prettier */ | ||||||||||||||||||||||||||||||||||||
/* eslint-disable no-unused-vars */ | ||||||||||||||||||||||||||||||||||||
import { useState } from 'react'; | ||||||||||||||||||||||||||||||||||||
import { motion } from 'framer-motion'; | ||||||||||||||||||||||||||||||||||||
import { useInView } from 'react-intersection-observer'; | ||||||||||||||||||||||||||||||||||||
import chess from '../../assets/img/chess.gif'; | ||||||||||||||||||||||||||||||||||||
import { FaStar } from 'react-icons/fa6'; | ||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove unused import The -import { FaStar } from 'react-icons/fa6'; 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
const ContactUs = () => { | ||||||||||||||||||||||||||||||||||||
const { ref, inView } = useInView({ | ||||||||||||||||||||||||||||||||||||
threshold: 0.2, | ||||||||||||||||||||||||||||||||||||
triggerOnce: true, | ||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
const animationVariants = { | ||||||||||||||||||||||||||||||||||||
hidden: { opacity: 0, y: 50 }, | ||||||||||||||||||||||||||||||||||||
visible: { opacity: 1, y: 0, transition: { duration: 0.5 } }, | ||||||||||||||||||||||||||||||||||||
}; | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
// Use an environment variable for backend URL | ||||||||||||||||||||||||||||||||||||
const API_URL = import.meta.env.VITE_BACKEND_URL || 'http://localhost:3000'; | ||||||||||||||||||||||||||||||||||||
const [mail, setMail] = useState(''); | ||||||||||||||||||||||||||||||||||||
const [subject, setSubject] = useState(''); | ||||||||||||||||||||||||||||||||||||
const [message, setMessage] = useState(''); | ||||||||||||||||||||||||||||||||||||
const [submitted, setSubmitted] = useState(false); | ||||||||||||||||||||||||||||||||||||
const [hover, setHover] = useState(null); | ||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove unused state variable The -const [hover, setHover] = useState(null); 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||
const [error, setError] = useState(null); | ||||||||||||||||||||||||||||||||||||
const [isLoading, setIsLoading] = useState(false); | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
const handleSubmit = async (e) => { | ||||||||||||||||||||||||||||||||||||
e.preventDefault(); | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
// Basic client-side validation for security | ||||||||||||||||||||||||||||||||||||
if (!mail || !subject || !message) { | ||||||||||||||||||||||||||||||||||||
setError('All fields are required, including the rating.'); | ||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Update error message to reflect actual fields. The error message mentions "including the rating," but there is no rating field in the form. Please update the message to accurately reflect the required fields. Apply this diff to correct the error message: - setError('All fields are required, including the rating.');
+ setError('All fields are required.'); 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||
return; | ||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
// Clear any previous errors | ||||||||||||||||||||||||||||||||||||
setError(null); | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
setIsLoading(true); | ||||||||||||||||||||||||||||||||||||
try { | ||||||||||||||||||||||||||||||||||||
const response = await fetch(`${API_URL}/api/contact/contactus`, { | ||||||||||||||||||||||||||||||||||||
method: 'POST', | ||||||||||||||||||||||||||||||||||||
headers: { | ||||||||||||||||||||||||||||||||||||
'Content-Type': 'application/json', | ||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||
body: JSON.stringify({ mail, subject, message }), | ||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add response status check after the fetch request. Currently, the code assumes that the fetch request is always successful. It's good practice to check the response status and handle errors accordingly. Apply this diff to check the response status: const response = await fetch(`${API_URL}/api/contact/contactus`, {
method: 'POST',
headers: {
'Content-Type': 'application/json',
},
body: JSON.stringify({ mail, subject, message }),
});
+ if (!response.ok) {
+ throw new Error('Network response was not ok');
+ }
setSubmitted(true); 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
setSubmitted(true); | ||||||||||||||||||||||||||||||||||||
setTimeout(() => { | ||||||||||||||||||||||||||||||||||||
setMail(''); | ||||||||||||||||||||||||||||||||||||
setSubject(''); | ||||||||||||||||||||||||||||||||||||
setMessage(''); | ||||||||||||||||||||||||||||||||||||
setSubmitted(false); | ||||||||||||||||||||||||||||||||||||
}, 3000); | ||||||||||||||||||||||||||||||||||||
} catch (error) { | ||||||||||||||||||||||||||||||||||||
setError('An error occurred while sending Mail...'); | ||||||||||||||||||||||||||||||||||||
console.error('Mail sending failed : ', error); | ||||||||||||||||||||||||||||||||||||
} finally { | ||||||||||||||||||||||||||||||||||||
setIsLoading(false); | ||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||
}; | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
return ( | ||||||||||||||||||||||||||||||||||||
<div className="bg-amber-100 h-full py-24 px-4 sm:px-6 lg:px-8"> | ||||||||||||||||||||||||||||||||||||
<div className="max-w-7xl mx-auto"> | ||||||||||||||||||||||||||||||||||||
<motion.div | ||||||||||||||||||||||||||||||||||||
ref={ref} | ||||||||||||||||||||||||||||||||||||
initial="hidden" | ||||||||||||||||||||||||||||||||||||
animate={inView ? 'visible' : 'hidden'} | ||||||||||||||||||||||||||||||||||||
variants={animationVariants} | ||||||||||||||||||||||||||||||||||||
className="lg:grid lg:grid-cols-2 lg:gap-8 lg:items-center" | ||||||||||||||||||||||||||||||||||||
> | ||||||||||||||||||||||||||||||||||||
<div className="mt-8 mb-8 lg:mb-0 relative"> | ||||||||||||||||||||||||||||||||||||
<h2 className="text-5xl font-black text-[#004D43]"> | ||||||||||||||||||||||||||||||||||||
Feel Free To Mail Us.. | ||||||||||||||||||||||||||||||||||||
</h2> | ||||||||||||||||||||||||||||||||||||
<p className="mt-5 text-lg text-gray-700 pb-3"> | ||||||||||||||||||||||||||||||||||||
Have questions or need assistance ? Reach out to us, and we'll be | ||||||||||||||||||||||||||||||||||||
happy to help !! | ||||||||||||||||||||||||||||||||||||
</p> | ||||||||||||||||||||||||||||||||||||
<div className="flex md:h-[40vh] md:w-[60vh] ml-20 mt-20 items-center justify-center mt-12"> | ||||||||||||||||||||||||||||||||||||
<img | ||||||||||||||||||||||||||||||||||||
src={chess} | ||||||||||||||||||||||||||||||||||||
alt="Chess" | ||||||||||||||||||||||||||||||||||||
loading="lazy" | ||||||||||||||||||||||||||||||||||||
className="md:p-10 p-5 object-contain bg-[#004D43] rounded-full shadow-2xl" | ||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
<div className="bg-[#004D43] rounded-xl p-3 pt-4 mt-40 h-fit"> | ||||||||||||||||||||||||||||||||||||
<form onSubmit={handleSubmit} className="space-y-4"> | ||||||||||||||||||||||||||||||||||||
<div> | ||||||||||||||||||||||||||||||||||||
<input | ||||||||||||||||||||||||||||||||||||
type="mail" | ||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Correct the input type for the email field. The Apply this diff to fix the input type: - type="mail"
+ type="email" 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||
id="mail" | ||||||||||||||||||||||||||||||||||||
value={mail} | ||||||||||||||||||||||||||||||||||||
placeholder="Email ID" | ||||||||||||||||||||||||||||||||||||
onChange={(e) => setMail(e.target.value)} | ||||||||||||||||||||||||||||||||||||
required | ||||||||||||||||||||||||||||||||||||
className="mt-1 block w-full border border-gray-300 rounded-md shadow-sm py-2 px-3 focus:outline-none focus:ring-[#004D43] focus:border-[#004D43]" | ||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
<div> | ||||||||||||||||||||||||||||||||||||
<input | ||||||||||||||||||||||||||||||||||||
type="text" | ||||||||||||||||||||||||||||||||||||
id="text" | ||||||||||||||||||||||||||||||||||||
placeholder="Subject" | ||||||||||||||||||||||||||||||||||||
value={subject} | ||||||||||||||||||||||||||||||||||||
onChange={(e) => setSubject(e.target.value)} | ||||||||||||||||||||||||||||||||||||
required | ||||||||||||||||||||||||||||||||||||
className="mt-1 block w-full border border-gray-300 rounded-md shadow-sm py-2 px-3 focus:outline-none focus:ring-[#004D43] focus:border-[#004D43]" | ||||||||||||||||||||||||||||||||||||
/> | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
<div> | ||||||||||||||||||||||||||||||||||||
<textarea | ||||||||||||||||||||||||||||||||||||
id="message" | ||||||||||||||||||||||||||||||||||||
placeholder="Write your message..." | ||||||||||||||||||||||||||||||||||||
rows="6" | ||||||||||||||||||||||||||||||||||||
value={message} | ||||||||||||||||||||||||||||||||||||
onChange={(e) => setMessage(e.target.value)} | ||||||||||||||||||||||||||||||||||||
required | ||||||||||||||||||||||||||||||||||||
className="mt-1 block w-full border border-gray-300 rounded-md shadow-sm py-2 px-3 focus:outline-none focus:ring-[#004D43] focus:border-[#004D43] resize-none" | ||||||||||||||||||||||||||||||||||||
></textarea> | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
<div> | ||||||||||||||||||||||||||||||||||||
<button | ||||||||||||||||||||||||||||||||||||
type="submit" | ||||||||||||||||||||||||||||||||||||
className="w-full flex justify-center py-2 px-4 border border-transparent rounded-md shadow-sm text-sm font-medium text-white bg-[#09342e] hover:bg-[#072d28] focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-[#004D43]" | ||||||||||||||||||||||||||||||||||||
disabled={isLoading} | ||||||||||||||||||||||||||||||||||||
> | ||||||||||||||||||||||||||||||||||||
{isLoading ? 'Sending...' : 'Send Mail'} | ||||||||||||||||||||||||||||||||||||
</button> | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
</form> | ||||||||||||||||||||||||||||||||||||
{submitted && ( | ||||||||||||||||||||||||||||||||||||
<motion.div | ||||||||||||||||||||||||||||||||||||
initial={{ opacity: 0, y: -10, display: 'none', height: 0 }} | ||||||||||||||||||||||||||||||||||||
animate={{ opacity: 1, y: 0, display: 'block', height: 'auto' }} | ||||||||||||||||||||||||||||||||||||
className="mt-4 p-4 bg-green-100 border border-green-400 text-green-700 rounded" | ||||||||||||||||||||||||||||||||||||
> | ||||||||||||||||||||||||||||||||||||
Thank you, We will reply you soon... | ||||||||||||||||||||||||||||||||||||
</motion.div> | ||||||||||||||||||||||||||||||||||||
)} | ||||||||||||||||||||||||||||||||||||
{error && ( | ||||||||||||||||||||||||||||||||||||
<motion.div | ||||||||||||||||||||||||||||||||||||
initial={{ opacity: 0, y: -10 }} | ||||||||||||||||||||||||||||||||||||
animate={{ opacity: 1, y: 0 }} | ||||||||||||||||||||||||||||||||||||
className="mt-4 p-4 bg-red-100 border border-red-400 text-red-700 rounded" | ||||||||||||||||||||||||||||||||||||
> | ||||||||||||||||||||||||||||||||||||
{error} | ||||||||||||||||||||||||||||||||||||
</motion.div> | ||||||||||||||||||||||||||||||||||||
)} | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
</motion.div> | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||
); | ||||||||||||||||||||||||||||||||||||
}; | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
export default ContactUs; |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codebase verification
Environment variables EMAIL_USER and EMAIL_PASS are not set.
Please set
EMAIL_USER
andEMAIL_PASS
in your deployment environment to prevent authentication issues.🔗 Analysis chain
Ensure environment variables are loaded correctly.
While you have
require("dotenv").config();
at the top, it's important to ensure that the environment variablesEMAIL_USER
andEMAIL_PASS
are properly loaded and handled in case they're missing.Run the following script to check if the environment variables are set:
Ensure that these variables are set in your deployment environment to prevent authentication errors.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 225