-
Notifications
You must be signed in to change notification settings - Fork 271
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
Added Subscription Boxes, Product Recommendations, Mood Board, and Vintage Storytelling #893
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for vintagevibe ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Great job, @sadneya145! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.We will promptly review your changes and offer feedback. Keep up the excellent work!
<meta charset="UTF-8" /> | ||
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
<title>RETRO Mood Board</title> | ||
<script src="https://cdnjs.cloudflare.com/ajax/libs/html2canvas/1.4.1/html2canvas.min.js"></script> |
Check warning
Code scanning / CodeQL
Inclusion of functionality from an untrusted source Medium
|
||
if (userMessage === '') return; | ||
|
||
chatBox.innerHTML += `<p><strong>You:</strong> ${userMessage}</p>`; |
Check warning
Code scanning / CodeQL
DOM text reinterpreted as HTML Medium
DOM text
router.get("/productget", async (req, res) => { | ||
try { | ||
const products = await Product.find(); | ||
if (products.length === 0) { | ||
return res.status(404).json({ error: "No products found" }); | ||
} | ||
res.json(products); | ||
} catch (error) { | ||
console.error("Error fetching products:", error); | ||
res.status(500).json({ error: "Internal Server Error" }); | ||
} | ||
}); |
Check failure
Code scanning / CodeQL
Missing rate limiting High
a database access
router.get("/productget/:id", async (req, res) => { | ||
try { | ||
const product = await Product.findById(req.params.id); | ||
if (!product) { | ||
return res.status(404).json({ error: "Product not found" }); | ||
} | ||
res.json(product); | ||
} catch (error) { | ||
console.error("Error fetching product:", error); | ||
res.status(500).json({ error: "Invalid Product ID or Internal Server Error" }); | ||
} | ||
}); |
Check failure
Code scanning / CodeQL
Missing rate limiting High
a database access
@sadneya145 all the these pages are linked in the main pages of our website right? |
Yes all pages are linked to main page except mood board page.
…On Thu, 13 Feb 2025, 8:32 pm Lakshmi Pavananjali, ***@***.***> wrote:
@sadneya145 <https://github.com/sadneya145> all the these pages are
linked in the main pages of our website right?
—
Reply to this email directly, view it on GitHub
<#893 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BCOPN63UYE63XVJOAM7W6GD2PSXXVAVCNFSM6AAAAABW3HH2A2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNJWHA3TMMBSGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
[image: Anjaliavv51]*Anjaliavv51* left a comment (Anjaliavv51/Retro#893)
<#893 (comment)>
@sadneya145 <https://github.com/sadneya145> all the these pages are
linked in the main pages of our website right?
—
Reply to this email directly, view it on GitHub
<#893 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BCOPN63UYE63XVJOAM7W6GD2PSXXVAVCNFSM6AAAAABW3HH2A2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNJWHA3TMMBSGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
y except mood board page? |
Added Subscription Boxes, Product Recommendations, Mood Board, and Vintage Storytelling
Description
This PR introduces four new features to enhance the user experience on RETRO:
These features aim to improve user engagement, retention, and sales.
Related Issues
Type of PR
Screenshots / videos (if applicable)
Screen.Recording.2025-02-10.232718.1.1.1.mp4
Checklist
main
branch before making this PRAdditional context: