Skip to content

bug fix:updated root page meta description - #798

Merged
cevheri merged 7 commits into
libredb:mainfrom
Dharshni-gth:fix-678-meta-description
Sep 12, 2026
Merged

bug fix:updated root page meta description#798
cevheri merged 7 commits into
libredb:mainfrom
Dharshni-gth:fix-678-meta-description

Conversation

@Dharshni-gth

Copy link
Copy Markdown
Contributor

Description

Updates the root page meta description to better represent LibreDB Studio's support for SQL and NoSQL databases without listing a fixed number of database engines.

Type of Change

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

Related Issue

Closes #678

Changes Made

  • Updated the root page meta description in src/app/layout.tsx.
  • Replaced the outdated list of specific database engines with a broader SQL and NoSQL description.
  • Updated the description to better reflect the database management, schema exploration, query, data import, and AI-assisted capabilities.

Testing

  • git diff --check` passed

  • bun run test completed, but the full test suite had unrelated existing failures (478 failed, 6 errors)

  • I have tested this locally

  • I have added/updated tests

  • All existing tests pass

Test Environment

  • LibreDB Studio Version: Development build
  • Browser: Google Chrome
  • OS: Windows
  • Node.js/Bun Version: Node.js v24.14.0 / Bun 1.4.2
  • Database Type: NA-metadata-change only

Screenshots (if applicable)

Checklist

  • My code follows the project's code 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 accordingly
  • 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
  • The required CI test job passes the 100% line-coverage gate (bun run test:coverage and bun run coverage:check)
  • If I changed src/lib/db/providers/, I updated the matching docs/providers/ documentation and tests/integration/db/ tests in the same PR (provider triad)
  • Any dependent changes have been merged and published

Additional Notes

This PR only updates the root page meta description in src/app/layout.tsx.
bun run test was executed, but the full test suite reported 478 failures and 6 errors. No failure was identified as related to this metadata-only change.

@cevheri cevheri added documentation Improvements or additions to documentation enhancement New feature or request labels Sep 11, 2026
@cevheri

cevheri commented Sep 11, 2026

Copy link
Copy Markdown
Member

thanks and welcome, I just fixed conflict
merging shortly

@Dharshni-gth

Copy link
Copy Markdown
Contributor Author

thank you for accepting my pull request

@cevheri
cevheri force-pushed the fix-678-meta-description branch from bdeaf43 to 147f65f Compare September 12, 2026 02:07
@codecov

codecov Bot commented Sep 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@cevheri
cevheri merged commit d2db8f5 into libredb:main Sep 12, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Root page's meta description still names four engines out of sixteen

2 participants