Skip to content

removing node_modules from backend and updated .gitignore and CustomerReview Section Fix#216

Closed
ayushpandey101 wants to merge 0 commit into
Skrishna0703:mainfrom
ayushpandey101:main
Closed

removing node_modules from backend and updated .gitignore and CustomerReview Section Fix#216
ayushpandey101 wants to merge 0 commit into
Skrishna0703:mainfrom
ayushpandey101:main

Conversation

@ayushpandey101

@ayushpandey101 ayushpandey101 commented Sep 27, 2025

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

The node_modules folder in the backend was not included in .gitignore, causing unnecessary Git tracking of dependency files. This leads to:

  • Frequent unwanted commits/changes in node_modules
  • Merge conflicts during collaboration
  • Bloated repository size
  • Inconsistent environments across machines

By removing node_modules from version control and adding it to .gitignore, we ensure:
✅ Cleaner commits
✅ Smaller repo size
✅ Consistent dependency installation via package.json + npm install
✅ Better team collaboration & Git hygiene

2nd issue resolved the Review-Content-Card Overflow
✅ Cleaner UI
✅ Arrow Changed
✅ Added Glow to swriper-btn

What changes are included in this PR?

Removed existing node_modules/ folder from Git tracking (if previously committed)
Added /node_modules/ to .gitignore in the backend root directory
Verified no other critical ignores were missing (optional: added common ones like .env, *.log if not present)

"⚠️ Note: Developers must run npm install after pulling this change to reinstall dependencies locally."

2nd issue changes include:

  • Fixed swiper navigation with modern design and glow effects
  • Updated pagination to orange theme with reduced glow
  • Removed inline styles from JSX, moved all styling to CSS
  • Fixed review card overflow issues and standardized heights
  • Added responsive design for mobile, tablet, and desktop
  • Implemented custom scrollbar for review comments"

Are these changes tested?

✅ Yes — manually verified:

  • Ran git status → confirms node_modules/ is no longer tracked
  • Cloned repo fresh → ran npm install → confirmed dependencies install correctly
  • Checked .gitignore syntax — valid and working

Are there any user-facing changes?

🚫 No.
This is a developer/Git workflow improvement. End users or app functionality are unaffected.

Screenshots (if Aplicable)

For issue #204
📸 Not applicable — this is a backend/Git configuration change with no UI impact.

For issue #206
Dark mode:
Screenshot 2025-10-01 185535

Light mode:
Screenshot 2025-10-01 185452

✅ Ready to merge.
Improves codebase maintainability and prevents future Git headaches.

@github-actions

Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@ayushpandey101

ayushpandey101 commented Sep 27, 2025

Copy link
Copy Markdown
Contributor Author

Hi @Skrishna0703,

I’ve removed the node_modules directory from version control and updated the backend .gitignore file to ensure it’s excluded from future commits. This prevents unnecessary bloat, avoids merge conflicts, and encourages consistent dependency management via package.json and npm install.

I've also included the changes of the other issue #206 with the issue #204 so that there is no git conflict issue while merging it to your codebase.

This change improves repository hygiene and streamlines collaboration for all contributors while also improving the CustomerReview Section .

Thanks!

@ayushpandey101 ayushpandey101 reopened this Oct 1, 2025
@ayushpandey101 ayushpandey101 changed the title removing node_modules from backend and updated .gitignore removing node_modules from backend and updated .gitignore and CustomerReview Section Fix Oct 1, 2025
@Skrishna0703 Skrishna0703 added GSSOC 25 Issues/PRs participating in GirlScript Summer of Code 2025 Level 1 Easy issue - 3 Points (GSSoC'25) labels Oct 3, 2025
@Skrishna0703

Copy link
Copy Markdown
Owner
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSSOC 25 Issues/PRs participating in GirlScript Summer of Code 2025 Level 1 Easy issue - 3 Points (GSSoC'25)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐞 Bug Report: Review Card Content Overflow(No Div) 🐞 Node_modules in Backend

2 participants