Skip to content

fix: update PostgreSQL configuration instructions for Docker environment - #65

Merged
ifrunruhin12 merged 6 commits into
masterfrom
fix/docs
Jan 12, 2026
Merged

fix: update PostgreSQL configuration instructions for Docker environment#65
ifrunruhin12 merged 6 commits into
masterfrom
fix/docs

Conversation

@TanvirTian

Copy link
Copy Markdown
Contributor

Description

This PR fixes and clarifies PostgreSQL configuration instructions in the Docker documentation.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring (no functional changes)
  • Performance improvement
  • Test updates

Changes Made

  • Clarified environment variable changes required after copying .env

Testing Checklist

  • Unit tests added or updated
  • Integration tests pass
  • Manual testing completed
  • Documentation updated (if applicable)
  • All existing tests pass

Code Quality Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation to reflect my changes
  • My changes generate no new warnings or errors
  • 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

Additional Context

N/A

@ifrunruhin12 ifrunruhin12 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change this and we are good to go

Comment thread docs/troubleshooting.md
Comment on lines +58 to +70
```bash
# ❌ Incorrect for Docker
DATABASE_URL=postgresql://postgres:password@localhost:5432/voice_collection

# ✅ Correct for Docker
DATABASE_URL=postgresql://postgres:password@postgres:5432/voice_collection

# ❌ Incorrect for Docker
REDIS_URL=redis://localhost:6379/0

# ✅ Correct for Docker
REDIS_URL=redis://redis:6379/0
```

@ifrunruhin12 ifrunruhin12 Jan 7, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove these emojis please. only saying correct and incorrect is enough. as you will notice our documentation has very minimal (close to none) emoji usage right now. some place where there is still irrelevant emoji we will remove most of it as well. but when adding new doc we won't be adding emojis anymore cause our docs was designed with visually appealing theme to begin with

@ifrunruhin12
ifrunruhin12 merged commit 807bead into master Jan 12, 2026
2 checks passed
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.

2 participants