-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
feat(poc): Disable widget based on country #6658
feat(poc): Disable widget based on country #6658
Conversation
✅ Deploy Preview for chatwoot-storybook canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍🏼
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
web: bundle exec rails ip_lookup:setup && bin/rails server -p $PORT -e $RAILS_ENV | ||
worker: bundle exec rails ip_lookup:setup && bundle exec sidekiq -C config/sidekiq.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could trigger this from the initializer. It would ensure that it stays consistent across all installation environments with out updating the documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sojan-official Can you please make the change if it is a small one?
Co-authored-by: Pranav Raj S <pranav@chatwoot.com>
Co-authored-by: Pranav Raj S <pranav@chatwoot.com>
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
https://www.loom.com/share/f2363462485b4aa8a6826f77a8964bf7
Changes:
The database download has been moved to a rake task and will be downloaded during boot up. Dynamically fetching (async fetches via Sidekiq) the database does not work well with synchronous calls since each server requires a copy of the file.
I updated the Procfile to download the DB initially and then run the service afterwards. This change may increase the bootup time by 3-4 seconds, depending on network latency.
I removed the IP_LOOKUP_SERVICE variable because none of the other services will work without the lookup parameter. Therefore, managing this variable is pointless.