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

fix(worker): Update docker-compose.yml - #6777 #6784

Closed
wants to merge 1 commit into from

Conversation

paulwer
Copy link
Contributor

@paulwer paulwer commented Oct 28, 2024

What changed? Why was the change needed?

According to #6777 the missing env resulted in an unwanted behavior and an strange error message.
This will fix this issue.

Special notes for your reviewer

#6777

@paulwer paulwer requested a review from a team as a code owner October 28, 2024 10:23
@paulwer paulwer requested review from SokratisVidros and merrcury and removed request for a team October 28, 2024 10:23
Copy link

netlify bot commented Oct 28, 2024

👷 Deploy request for novu-stg-vite-dashboard-poc pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 5574818

@paulwer paulwer changed the title Update docker-compose.yml - #https://github.com/novuhq/novu/issues/6777 fix(worker): Update docker-compose.yml - #https://github.com/novuhq/novu/issues/6777 Oct 28, 2024
@paulwer
Copy link
Contributor Author

paulwer commented Oct 28, 2024

@SokratisVidros @merrcury the error message seen in #6777 should be looked into seperatly.

@paulwer paulwer changed the title fix(worker): Update docker-compose.yml - #https://github.com/novuhq/novu/issues/6777 fix(worker): Update docker-compose.yml - #6777 Oct 28, 2024
@merrcury
Copy link
Member

We have a duplicate one open here. We should be be close this with discussion there. Thanks

@paulwer paulwer closed this Oct 29, 2024
@paulwer paulwer deleted the patch-1 branch October 29, 2024 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants