Skip to content

Qa/appointment#154

Merged
dashaaaa21 merged 6 commits intodevelopfrom
QA/appointment
Mar 8, 2026
Merged

Qa/appointment#154
dashaaaa21 merged 6 commits intodevelopfrom
QA/appointment

Conversation

@dashaaaa21
Copy link
Copy Markdown
Contributor

No description provided.

@dashaaaa21 dashaaaa21 requested review from Dmytro-Doronin and Copilot and removed request for Copilot March 8, 2026 17:18
@HackYourFutures HackYourFutures temporarily deployed to study-bridge-qa-appoint-05ulxe March 8, 2026 17:18 Inactive
@dashaaaa21 dashaaaa21 removed the request for review from Dmytro-Doronin March 8, 2026 17:21
Copilot AI review requested due to automatic review settings March 8, 2026 17:22
@HackYourFutures HackYourFutures temporarily deployed to study-bridge-qa-appoint-7bfmtu March 8, 2026 17:23 Inactive
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 updates build/start entrypoints for the server, adjusts Heroku build scripting, moves client build tooling dependencies, and adds a new Jest test suite covering student query/command/model behaviors.

Changes:

  • Update server start/main entrypoint to dist/src/index.js (matching tsc output structure).
  • Add a comprehensive StudentQuery/StudentCommand/StudentModel Jest test suite.
  • Adjust root heroku-postbuild flow and move Vite/TypeScript tooling into client runtime dependencies.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File Description
server/package.json Points main and start to the compiled output path under dist/src/.
server/tests/student/student.service.test.ts Adds student repository/model CRUD and update behavior tests.
package.json Changes heroku-postbuild script sequence.
client/package.json Moves Vite/TypeScript/plugin-react from devDependencies to dependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@HackYourFutures HackYourFutures temporarily deployed to study-bridge-qa-appoint-wmcmqb March 8, 2026 17:30 Inactive
@HackYourFutures HackYourFutures temporarily deployed to study-bridge-qa-appoint-ptakya March 8, 2026 17:37 Inactive
@HackYourFutures HackYourFutures temporarily deployed to study-bridge-qa-appoint-p34ggf March 8, 2026 17:40 Inactive
@dashaaaa21 dashaaaa21 merged commit 27d7655 into develop Mar 8, 2026
2 checks passed
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.

4 participants