Feature/admin #224

samriddhitiwary · 2024-10-20T05:21:49Z

Admin.1.mp4

Pull Request Template

Title: Create ADMIN page to CREATE, READ, UPDATE, and DELETE Products into MongoDB using existing CRUD API

Description

Please provide a clear and concise description of what this pull request does, including any relevant details.

Related Issue

Link to the issue that this pull request addresses (if applicable). Use the format: Fixes #165

Changes Made

This issue involves building an Admin page that allows for the Create, Read, Update, and Delete (CRUD) operations on Products stored in MongoDB. The page will interact with an existing CRUD API for product management. The page should include a form for product creation and editing, a table to display all products, and functionality to delete products.

Checklist

I have tested my code and it works as expected.
I have added/updated documentation where necessary.
I have included tests for new features.
I have followed the project's coding standards.
Checklist:
- I have tested my code and it works as expected.
- I have added documentation for the changes made.
- I have included tests for the registration form validation.

netlify · 2024-10-20T05:22:13Z

✅ Deploy Preview for scrollme-ecommerce ready!

Name	Link
🔨 Latest commit	`81580d0`
🔍 Latest deploy log	https://app.netlify.com/sites/scrollme-ecommerce/deploys/6714a1f85b1a5f000848e555
😎 Deploy Preview	https://deploy-preview-224--scrollme-ecommerce.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

saurabhbakolia · 2024-10-20T05:50:10Z

Hey @samriddhitiwary ,
This is cool, I am reviewing it.

vercel · 2024-10-20T06:05:21Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
scrollme-ecommerce-website	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 20, 2024 6:25am

samriddhitiwary · 2024-10-20T06:06:06Z

@thakuratul2 I have resolved the conflicts

samriddhitiwary · 2024-10-20T06:26:46Z

@saurabhbakolia @thakuratul2 I have resolved the conflict, could you please review the PR

saurabhbakolia · 2024-10-20T06:12:31Z

src/components/UpdateProduct.jsx

+  `}
+</style>
+
+      <div className='container mt-5'>


Great work on the app! I'd suggest using styled-components for more modular and maintainable styling.

Sure, I am making these changes.

saurabhbakolia · 2024-10-20T06:15:20Z

backend/api/routes/productRoute.js

@@ -4,7 +4,7 @@ const authMiddleware = require('../middleware/authMiddleware');
 const productController = require('../controllers/productController');

 // Route to add a new product (only for admins)
-router.post('/add', authMiddleware, productController.addProduct);


Here you can add the support to edit this, only for admins and sellers.

As of now, I have disabled all the auth flow. Later, we can add it in separate issue and PR?

saurabhbakolia · 2024-10-20T06:15:45Z

backend/server.js

 app.use(cookieParser());
 app.use(helmet());

 app.use("/api/auth", authRoutes); // Authentication routes
 app.use("/api/cart", authMiddleware, cartRoute); // Cart routes
 app.use("/api/product", productRoute); // Product routes
 app.use("/api/user", userRoute); // user routes
-app.use("/api/auth", authStatusRoute); // Check Auth Status


Why this change?

@saurabhbakolia
I am sorry, but, for time being, can we hold all Auth related flow, Its causing lots of issue in local. We can have separate issue to fix all the auth related work.

saurabhbakolia · 2024-10-20T06:16:21Z

backend/server.js

-    console.log(`Server running on port ${PORT}`);
-    console.log(`API available at http://localhost:${PORT}/api/auth`);
-  });
+// const PORT = process.env.PORT || 8080;


why this change ?

I am trying to connect the MongoDB ATLAS. It seems, existing code was not compatible with ATLAS, So when i modified, It started working. I did not try with local mongo db.

saurabhbakolia · 2024-10-20T06:17:15Z

src/App.js

@@ -10,6 +10,10 @@ import ProtectedRoute from './utils/ProtectedRoute';
 import ContributorsPage from './pages/ContributorsPage';
 import { lazy, Suspense } from 'react';
 import ContactUsPage from './pages/ContactUsPage';
+import Admin from './components/Admin';


Have you installed prettier and eslint in your IDE?

Sure, I will do it. I missed it.

saurabhbakolia · 2024-10-20T06:20:03Z

package.json

@@ -13,11 +13,13 @@
    "@testing-library/react": "^13.4.0",
    "@testing-library/user-event": "^13.5.0",
    "axios": "^1.6.0",
+    "bootstrap": "^5.3.3",


Why did you install Bootstrap? We are using styled-components for styling and material-ui-icons for icons. Could you please stick to these for now?

I will check, I am not using anywhere. I will remove it.

saurabhbakolia · 2024-10-20T06:20:54Z

package.json

    "firebase": "^10.14.0",
    "framer-motion": "^11.9.0",
    "myapp": "file:",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
+    "react-hot-toast": "^2.4.1",


You don't need to use react-hot-toast; we're using chakra-ui, so please use their toast method instead.

I will check and will use chakra-ui.

saurabhbakolia · 2024-10-20T06:22:00Z

src/App.js

@@ -42,14 +46,6 @@ function App() {
            </Suspense>
          }
        />
-        <Route


I think you didn't pull the latest changes. Please avoid removing code unnecessarily.

saurabhbakolia · 2024-10-20T06:22:30Z

src/components/AddNewProduct.jsx

+          background: `linear-gradient(rgba(255, 255, 255, 0.5), rgba(255, 255, 255, 0.5)), url('https://images.pexels.com/photos/6984650/pexels-photo-6984650.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940') center`,
+          backgroundSize: 'cover',
+          backgroundAttachment: 'fixed', // Keep the background fixed during scroll
+          minHeight: '100vh', // Ensure it covers the full height of the viewport


Remove unnecessary comments and logs.

Sure, will do it.

samriddhitiwary · 2024-10-20T10:56:49Z

@saurabhbakolia Ok I will check it again

saurabhbakolia · 2024-10-25T14:16:32Z

@saurabhbakolia Ok I will check it again

Hey @samriddhitiwary,

if you can make the required changes so that I can merge this PR.

samriddhitiwary and others added 8 commits October 19, 2024 12:00

Trial2

c30ff2d

Admin changes2

5a6943b

Conflicts

864629e

Add Product Buttodan

c686a13

Del1

2ee19d2

Final Changes

9a86bbf

Admin Page Created

384462d

Fixed Bugs

b1f6ec2

saurabhbakolia self-requested a review October 20, 2024 05:49

Fixed Conflicts3

b0f75aa

vercel bot deployed to Preview October 20, 2024 06:05 View deployment

Removed Conflicts

81580d0

vercel bot deployed to Preview October 20, 2024 06:25 View deployment

thakuratul2 self-requested a review October 20, 2024 06:30

saurabhbakolia requested changes Oct 20, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/admin #224

Feature/admin #224

samriddhitiwary commented Oct 20, 2024

netlify bot commented Oct 20, 2024 •

edited

Loading

saurabhbakolia commented Oct 20, 2024

vercel bot commented Oct 20, 2024 •

edited

Loading

samriddhitiwary commented Oct 20, 2024

samriddhitiwary commented Oct 20, 2024 •

edited

Loading

saurabhbakolia Oct 20, 2024

samriddhitiwary Oct 20, 2024

saurabhbakolia Oct 20, 2024

samriddhitiwary Oct 20, 2024

saurabhbakolia Oct 20, 2024

samriddhitiwary Oct 20, 2024 •

edited

Loading

saurabhbakolia Oct 20, 2024

samriddhitiwary Oct 20, 2024

saurabhbakolia Oct 20, 2024

samriddhitiwary Oct 20, 2024

saurabhbakolia Oct 20, 2024

samriddhitiwary Oct 20, 2024

saurabhbakolia Oct 20, 2024

samriddhitiwary Oct 20, 2024

saurabhbakolia Oct 20, 2024

saurabhbakolia Oct 20, 2024

samriddhitiwary Oct 20, 2024

samriddhitiwary commented Oct 20, 2024

saurabhbakolia commented Oct 25, 2024

Feature/admin #224

Are you sure you want to change the base?

Feature/admin #224

Conversation

samriddhitiwary commented Oct 20, 2024

Pull Request Template

Description

Related Issue

Changes Made

Checklist

netlify bot commented Oct 20, 2024 • edited Loading

✅ Deploy Preview for scrollme-ecommerce ready!

saurabhbakolia commented Oct 20, 2024

vercel bot commented Oct 20, 2024 • edited Loading

samriddhitiwary commented Oct 20, 2024

samriddhitiwary commented Oct 20, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

samriddhitiwary Oct 20, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

samriddhitiwary commented Oct 20, 2024

saurabhbakolia commented Oct 25, 2024

netlify bot commented Oct 20, 2024 •

edited

Loading

vercel bot commented Oct 20, 2024 •

edited

Loading

samriddhitiwary commented Oct 20, 2024 •

edited

Loading

samriddhitiwary Oct 20, 2024 •

edited

Loading