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

MTK: update compatibility content #6438

Open
wants to merge 19 commits into
base: develop
Choose a base branch
from

Conversation

gvasquezvargas
Copy link
Contributor

@gvasquezvargas gvasquezvargas commented Jan 29, 2025

What Changed?

https://enterprisedb.atlassian.net/browse/DOCS-1154

Changes:

  • Added a section on compatibility page that talks about “Supported JDBC drivers”. It is divided in two subsections:
    • Supported postgres JDBC drivers, which includes a table with MTK and EDB JDBC/community JDBC compatibility.
    • Supported non-postgres drivers, which explains the JDBC version depends on the database its supposed to migrate from or to.
  • Improved the “installing a JDBC driver” page by replacing the text which was a bit confusing with a table where you can fetch the direct download links to JDBC drivers according to your db.
  • Added a subsection “Driver version compatibility” where I suggest always getting the latest MTK, and if not, ensuring driver compatibility.
  • Fixed the order of the left navi topics under "Installing"

@gvasquezvargas gvasquezvargas force-pushed the mtk/compatibility-updates branch from 97d8ac9 to d463dbe Compare January 31, 2025 12:59
@gvasquezvargas gvasquezvargas marked this pull request as ready for review January 31, 2025 12:59
@gvasquezvargas gvasquezvargas requested a review from a team as a code owner January 31, 2025 12:59
@gvasquezvargas gvasquezvargas added the deploy Add this label to a PR and it will automatically be deployed to netlify label Jan 31, 2025
Copy link
Contributor

github-actions bot commented Jan 31, 2025

@github-actions github-actions bot temporarily deployed to pull request January 31, 2025 13:26 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 4, 2025 14:08 Inactive
| SAP Adaptive Server Enterprise (ASE) | [Sybase ASE (jTDS) JDBC driver](https://sourceforge.net/projects/jtds/files/) (See notes) |

!!!note Notes
- The EDB JDBC connector is not available for Mac OS.
Copy link

Choose a reason for hiding this comment

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

The "Mac OS" is no more being used for modern Apple desktop systems, rather "macOS" is used. For consistency (as in some other places in the doc, "macOS" is used), better to use "macOS".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zahid75 sounds good, I did a global search of all instances in the mtk docs and replaced them for consistency. Could you please take another look?

Copy link

Choose a reason for hiding this comment

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

Suggest change in a couple of places.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey @zahid75 , I could only find your suggestion about dropping the X on macOS mentions. I have done a search+replace of that. Was there anything else pending?

@github-actions github-actions bot temporarily deployed to pull request February 5, 2025 07:53 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 5, 2025 08:05 Inactive
Copy link

@zahid75 zahid75 left a comment

Choose a reason for hiding this comment

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

Change required in a couple of places, rest looks fine.

gvasquezvargas and others added 10 commits February 6, 2025 10:41
…rity fields

Signed-off-by: Dj Walker-Morgan <dj.walker-morgan@enterprisedb.com>
Also include for meta
Also enable edit link to use this information
Also use correct capitalization for originalFilePath
- edit links
- issues
- history

Move as much as possible into components and static queries; avoid static rendering hashes and feature branch names. Should slightly reduce build size, massively reduce differences between deploys.
@github-actions github-actions bot temporarily deployed to pull request February 6, 2025 09:47 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 6, 2025 09:56 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 6, 2025 10:27 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Add this label to a PR and it will automatically be deployed to netlify
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants