Skip to content

Conversation

martakam
Copy link
Member

This PR adds environment variables for deployment and documentation on how to deploy the app to the VM we have been provided.

@martakam martakam self-assigned this Oct 15, 2026
@martakam martakam added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 15, 2026
@martakam martakam added this to the Deliverable 2 milestone Oct 15, 2026
@martakam martakam linked an issue Oct 15, 2026 that may be closed by this pull request
@martakam martakam marked this pull request as ready for review October 15, 2026 14:18
@martakam martakam requested a review from a team as a code owner October 15, 2026 14:18
@johahor
Copy link
Member

johahor commented Oct 15, 2026

I've now added documentation for Node, pnpm and PostgreSQL and how that was installed. I also added the Apache FallBackResource config on the VM (look documentation for why).

In the documentation under "Build the frontend" the current command only works the first time it is setup. The second time project2 already exsists, and mv puts the new build inside it as project2/client/, and the site keeps serving the old version. As far as i can see this has already happened, since there is a client/ folder inside /var/www/html/project2.

Suggested replacment is these two commands:

sudo rm -rf /var/www/html/project2
sudo cp -r build/client /var/www/html/project2

I'll add these as suggested change, so if you agree you can just confirm them.

Co-authored-by: Johannes Horn <johahor@stud.ntnu.no>
@johahor
Copy link
Member

johahor commented Oct 15, 2026

Could you deploy the frontend again with the new commands, and that checks two things:

That the commands are working, and the favicon changes are working.

Let me know when that is done, and I'll approve the PR

@martakam
Copy link
Member Author

Yup, everything seems to be good - ran all the frontend commands now

@johahor
Copy link
Member

johahor commented Oct 15, 2026

Perfect. Checked the deployed version, and the errors in the console is gone. Approving.

Sign in to join this conversation on GitHub.
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore: deploy database, backend and frontend to the VM
2 participants