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

Refactor and code update for a working docker deployment #17

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

aperera
Copy link

@aperera aperera commented May 23, 2024

Refactor and update to run worklenz-frontend in the docker environment.

  1. docker-compose.yml
  2. worklenz-frontend/package.json
  3. worklenz-frontend/Dockerfile

@CLAassistant
Copy link

CLAassistant commented May 23, 2024

CLA assistant check
All committers have signed the CLA.

@kalinga777
Copy link
Contributor

Thanks @aperera for your contribution. We will review and merge.

@gdevenyi
Copy link
Contributor

I recommend #20 instead of this.

@aperera aperera changed the title Refactor and update | frontend for docker Refactor and code update for a working deployment May 24, 2024
@aperera aperera changed the title Refactor and code update for a working deployment Refactor and code update for a working docker deployment May 24, 2024
@aperera
Copy link
Author

aperera commented May 24, 2024

  1. Moved .editorconfig to root Avoids confusion in settings.
  2. Enhanced Documentation Added DOCKER_SETUP.md for detailed Docker deployment instructions.
  3. Updated docker-compose.yml Supports latest changes.
  4. Upgraded Postgres to v16.3 Strengthened security and resolved v15.3 user authentication issue.
  5. Manual Database Migration Included process in DOCKER_SETUP.md and added PGAdmin for GUI management.
  6. Updated Dockerfiles Switched to Ubuntu:23.04 for both frontend and backend; production-ready and ease of troubleshooting.
  7. Updated 6_user-permission.sql Reflected new database settings.
  8. Minor Refactoring Additional settings and file updates.

These changes will help others to get up and running in a few minutes.

@gdevenyi
Copy link
Contributor

This PR has huge non-isolated change which make it difficult to review, I suggest breaking it up to change only closely related things.

@aperera
Copy link
Author

aperera commented May 24, 2024

@gdevenyi Besides the .editorconfig and package.json URL updates, everything else is required to get the Docker environment up and working. I included the documentation since I could not find a migration tool in the current packages, but feel free to break it down into multiple PRs.

@gdevenyi
Copy link
Contributor

As per my PR, the changes are far less extensive to get the docker working. Only thing missing in mine is readme changes to change the databasename and password in the 6_usermanagement.sql

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