Skip to content

Conversation

asgeija
Copy link
Member

@asgeija asgeija commented Oct 15, 2026 •

Closes #41

What changed

  • README.md:
    • Rewrote "Running the Project" as steps from a fresh clone: install, database setup and import, backend, frontend.
    • Added a section for course staff on where the backend settings are on the VM and how to start the backend.
    • Replaced "Planned Functionality" with "Functionality", which describes what works now and what is still missing.
    • Updated the parts that still said "will be" or "not decided".
  • AGENTS.md: added the database setup commands, updated the folder structure, and added the backend settings and rollback to the Deployment section.
  • docs/ci-cd.md: fixed a line that said there were no API tests and that npm test skips the backend.
  • Also replaced the dead dataset link

Testing

  • npx prettier --check README.md AGENTS.md docs/ci-cd.md
  • I checked the commands and paths against the scripts and config files in the repo.

Rewrite the README so someone can run the project from a fresh clone:database setup and import, backend, frontend.

Replace "Planned Functionality"
with what works now and what is still missing.

Update AGENTS.md with the database commands, the folder structure and the backend settings on the VM. Fix docs/ci-cd.md, which still said there were no API tests.

Closes #41
@asgeija asgeija added the documentation Improvements or additions to documentation label Oct 15, 2026
@asgeija asgeija added this to the Partial submission 2 milestone Oct 15, 2026
@asgeija asgeija requested a review from a team as a code owner October 15, 2026 09:33
@asgeija asgeija linked an issue Oct 15, 2026 that may be closed by this pull request
4 tasks
Copy link
Member

@jonatsm jonatsm left a comment •

Choose a reason for hiding this comment

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

Overall good changes, some suggestions incoming:

Copy link
Member

@jonatsm jonatsm left a comment

Choose a reason for hiding this comment

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

Nice! Some feedback that should be addressed:

  1. Rollback only works back to #73. Earlier runs have no production deployment and no deploy job, and runs before #70 don't deploy at all. Add a note saying re-run only works for deploys from #73 onward, and use a revert for anything older.
  2. The site check also loads the start page. The text says it only tests the backend and the database, which contradicts the step table above it. Correct the sentence.
  3. "There are no API tests" is outdated (finding 7). #74 added API and database tests. Update the section.

@thortve thortve requested a review from jonatsm October 15, 2026 10:10
Copy link
Member

@jonatsm jonatsm left a comment

Choose a reason for hiding this comment

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

Looks good!

@jonatsm jonatsm merged commit 8897a33 into main Oct 15, 2026
2 checks passed
Sign in to join this conversation on GitHub.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Write setup and startup documentation
2 participants