# Pull Request Template
## Description
- Fix docker development
- Switch from webpack to vite
## Type of change
Please delete options that are not relevant.
- [x] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [x] Breaking change (fix or feature that would cause existing
functionality not to work as expected)
## How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration.
## Checklist:
- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my code
- [ ] I have commented on my code, particularly in hard-to-understand
areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] Any dependent changes have been merged and published in downstream
modules
POSTGRES_PORT was not taking effect if provided separately
instead of using DATABASE_URL. This adds support for using
databases running on non-standard ports.
#1145#1147
Co-authored-by: Pranav Raj S <pranav@chatwoot.com>
* fix: pg pass printed by docker container (#1371)
The "POSTGRES_PASSWORD" variable setting inside the entrypoint script
of the rails docker container was printed in the logs when the
container was run using docker compose.
Fixed this by removing this password being set in this script.
Added env file from root directory to the container
Updated the tailwind evrsion in yarn lock
* fix: moved pg database url check in the entrypoint to another helper
created a new helper docker/entrypoints/helpers/pg_database_url.sh
to parse the databse url and export the postgres host, user and
port.
With this the pg is_ready check falls to a common format rather
than two formats depending on DATABASE_URL env variable is
present or not.
* fix: rename pg ready variable in rails entrypoint
* update db:schema due to migration
* reduce default concurrency for redis connection limit
* Allow auto update packages in the development mode without building image
* add sidekiq support to docker-compose
* Pass sidekiq cofig file
* passed the env file in base image rather than separately in rails, web packer and sidekiq in docker-compose
* removed un-necessary changes in schema
* changed concurrency to finer values
* removed default size set in sidekiq redis config
* Added the sidekiq config option in Procfile.dev
Co-authored-by: Sony Mathew <ynos1234@gmail.com>
* updated development docker setup
* turned on yarn integrity check
* create test docker compose and update development dockerfile
* create docker Readme.md file
* fix README.md file
* create docker-copmose and dockerfile for production/staging environment setup
* added mailhog to catch email
* remove yarn integrity check in development mode!
* Update Readme.md file to support mailhog inbox
* update link to docker development guide from mail README
* remove .env.development and use .env.example for docker as .env.development was mixing with circleci config
* make the dockerfile configurable like before
* update docker-compose as per suggesion in PR
* create docker setup docs
* Revert accidental deletion
* fix: typo for branding consistency
* fix typo
* update the code as per recommendation in PR
* remove package-lock and add gitignore
* fix vulnerability AND update env for tests to pass
* Fix yarn integrity check across different docker services
* update code based on rubocop and PR suggestions
* remove redundant test docker-compose and dockerfile
* update rails entrypoint, update docs and Gemfile
* [Rubocop] order Gemfile alphabetically
* Reordering Gemfile