Skip to content

fixed multiple uris for no reason - #4

Open
AuroraAustralis7 wants to merge 1 commit into
stuyspec:mainfrom
AuroraAustralis7:one-uri
Open

AuroraAustralis7 wants to merge 1 commit into
stuyspec:mainfrom
AuroraAustralis7:one-uri

Conversation

@AuroraAustralis7

Copy link
Copy Markdown

No description provided.

Copilot AI lite review requested due to automatic review settings August 13, 2026 02:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR appears to consolidate MongoDB connection configuration to a single MONGODB_URI environment variable and applies minor formatting/cleanup changes across the server routes/middleware and the client lockfile.

Changes:

  • Switch model connection strings in staff/paper (and intended in article) to use process.env.MONGODB_URI.
  • Apply small style/formatting fixes in Express routers/middleware (whitespace, chaining layout, semicolon).
  • Update client/package-lock.json by removing some optional @emnapi/* entries.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
server/src/routes/pdfRouter.js Minor whitespace cleanup near export.
server/src/routes/articleRouter.js Reformats Mongoose query chaining and removes stray blank line.
server/src/routes/api.js Adds missing semicolon on router.use("/article", ...).
server/src/models/staff.js Switches DB connection env var to MONGODB_URI.
server/src/models/paper.js Switches DB connection env var to MONGODB_URI.
server/src/models/article.js Attempts to switch env var, but currently leaves a broken reference (see comment).
server/src/middlewares.js Normalizes indentation in exports.
client/package-lock.json Removes some optional dependency entries.
Files not reviewed (1)
  • client/package-lock.json: Generated file

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +4 to 5
const db_str = process.env.MONGODB_URI;
const articleConnection = mongoose.createConnection(db_str_articles);
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants