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

docs: add best practice section to contribution guide #4288

Conversation

swiffer
Copy link
Contributor

@swiffer swiffer commented Oct 20, 2024

this adds a best practice section to contribution guide:

  • usage of timestamp columns
  • enabling pg_stat_statements
  • usage of streaming api / positions table in dashboard queries

fixes #4269

Copy link

netlify bot commented Oct 20, 2024

Deploy Preview for teslamate ready!

Name Link
🔨 Latest commit 5dcb9e4
🔍 Latest deploy log https://app.netlify.com/sites/teslamate/deploys/67165b73dbe1060008f5bdf1
😎 Deploy Preview https://deploy-preview-4288--teslamate.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JakobLichterfeld JakobLichterfeld added the kind:documentation Adds or improves documentation label Oct 21, 2024
Copy link
Collaborator

@JakobLichterfeld JakobLichterfeld left a comment

Choose a reason for hiding this comment

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

Thanks a lot! One small finding

website/docs/development.md Outdated Show resolved Hide resolved
@JakobLichterfeld JakobLichterfeld merged commit 93fb64e into teslamate-org:master Oct 21, 2024
14 of 15 checks passed
@swiffer swiffer deleted the contribution-guide-additions-db-perf branch October 21, 2024 15:16
@@ -182,7 +182,7 @@ To quickly identify performance bottlenecks we encourage all contributors to ena
- Create Extension to enable `pg_stat_statements` view

```sql
CREATE EXTENSION IF NOT EXISTS `pg_stat_statements`;
CREATE EXTENSION IF NOT EXISTS pg_stat_statements;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@JakobLichterfeld - thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:documentation Adds or improves documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enabling pg_stat_statements in Docker Compose
2 participants