Update shared SQL database page for PostgreSQL-only live sync - #668
Update shared SQL database page for PostgreSQL-only live sync#668koppor wants to merge 6 commits into
Conversation
PR Summary by QodoDocument PostgreSQL-only shared databases and live sync
AI Description
Diagram
High-Level Assessment
Files changed (1)
|
Code Review by Qodo
1. Migration support is overstated
|
| To use this feature you have to connect to a remote database. To do so you have to open **File** in the menu bar and then click the **Connect to shared database** item. The **Connect to shared database** dialog will open and you will have to fill in the shared's database connection settings. Then, you have to fill out the remaining fields with the according information. If you like you can save your password by clicking the **Remember password?** checkbox. | ||
| To use this feature you have to connect to a remote database. To do so you have to open **File** in the menu bar and then click **Shared database** and **Connect to shared database**. The **Connect to shared database** dialog will open and you will have to fill in the shared's database connection settings. Then, you have to fill out the remaining fields with the according information. If you like you can save your password by clicking the **Remember password?** checkbox. | ||
|
|
||
| If the database was already used by an older version of JabRef, its content is migrated to the new table structure automatically when the first client connects. The old tables are kept untouched, so older JabRef versions can still be used with them. |
There was a problem hiding this comment.
1. Migration support is overstated 🐞 Bug ≡ Correctness
The new text says any database used by an older JabRef is migrated automatically and remains usable by older clients, but the documented pre-3.6 schema is explicitly unsupported and requires a manual export/import migration. Users with such databases may expect a safe first-connect migration that this documentation says is unavailable.
Agent Prompt
## Issue description
The automatic-migration statement applies without a version or schema boundary, contradicting the documented manual migration requirement for pre-3.6 databases.
## Issue Context
State exactly which prior JabRef versions/table layouts the new migration supports, and retain a clear exception or link for pre-3.6 databases. Reconcile the dedicated migration page if its workflow is no longer applicable.
## Fix Focus Areas
- en/collaborative-work/sqldatabase/README.md[13-13]
- en/collaborative-work/sqldatabase/sqldatabasemigration.md[5-24]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
| ## Usage | ||
|
|
||
| To use this feature you have to connect to a remote database. To do so you have to open **File** in the menu bar and then click the **Connect to shared database** item. The **Connect to shared database** dialog will open and you will have to fill in the shared's database connection settings. Then, you have to fill out the remaining fields with the according information. If you like you can save your password by clicking the **Remember password?** checkbox. | ||
| To use this feature you have to connect to a remote database. To do so you have to open **File** in the menu bar and then click **Shared database** and **Connect to shared database**. The **Connect to shared database** dialog will open and you will have to fill in the shared's database connection settings. Then, you have to fill out the remaining fields with the according information. If you like you can save your password by clicking the **Remember password?** checkbox. |
There was a problem hiding this comment.
2. Migration menu path stays stale 🐞 Bug ≡ Correctness
The new menu hierarchy is updated only on the main page, while the published pre-3.6 migration workflow still sends users to **File → Open shared database**. Users following that indexed workflow in a current JabRef version will be directed to a command replaced by **File → Shared database → Connect to shared database**.
Agent Prompt
## Issue description
The main page documents the new nested shared-database menu, but the dedicated migration instructions retain the prior command name.
## Issue Context
Update the migration workflow to distinguish the menu path required in historical JabRef 3.6 from the path in current JabRef, or replace the stale path if the workflow is intended for current clients.
## Fix Focus Areas
- en/collaborative-work/sqldatabase/README.md[11-11]
- en/collaborative-work/sqldatabase/sqldatabasemigration.md[20-23]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyZEicma9suBn7QJxAiZS1
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyZEicma9suBn7QJxAiZS1
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
Updates the shared SQL database page for the sync rework in JabRef PR JabRef/jabref#11879: PostgreSQL is the only supported system, changes from other users arrive live (the manual "pull changes" action is gone), the new menu path, and the automatic table migration on first connect.
This is a low-risk documentation update — review and merge can be quick. Ideally merged together with the JabRef PR.
🤖 Generated with Claude Code