Skip to content
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

modified .env.example,.gitignore,app.js #1309

Closed
wants to merge 1 commit into from

Conversation

VarnitRaina
Copy link

Summary
This pull request includes several improvements and fixes to enhance the functionality and maintainability of the application. Key changes include the addition of the uploads directory to .gitignore, improvements to the MongoDB connection handling, and updates to the rate limiter configuration.

Changes Made
Updated .gitignore:

Added the uploads/ directory to the .gitignore file to prevent uploaded files from being tracked in the repository.
Improved MongoDB Connection Handling:

Enhanced the MongoDB connection logic in app.js to include more informative console messages on connection success and error handling. This provides better visibility into the application’s status during development.
Increased Rate Limiter Configuration:

Adjusted the rate limiter settings to allow for increased requests per time window, improving the application's ability to handle traffic and reducing the likelihood of request throttling.
Additional Code Refactoring:

Made minor adjustments and optimizations in app.js for better readability and performance.

Copy link

@sravanth-space sravanth-space left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@YasharF
Copy link
Collaborator

YasharF commented Jan 23, 2025

These changes are not needed, they break the build.

@YasharF YasharF closed this Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants