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

Update README.md to include MariaDB #43343

Merged
merged 5 commits into from
Sep 2, 2024
Merged

Conversation

robertsilen
Copy link
Contributor

Considering the technical requirements (https://downloads.joomla.org/technical-requirements) mention MariaDB, I humbly suggest the README should include MariaDB too, i.e. MySQL/MariaDB.

Pull Request for Issue # .

Summary of Changes

Testing Instructions

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Considering the technical requirements (https://downloads.joomla.org/technical-requirements) mention MariaDB, I humbly suggest the README should include MariaDB too, i.e. MySQL/MariaDB.
@HLeithner
Copy link
Member

We no longer support, SQL Server, and we support MariaDB not official in 4.x. Started with Joomla 5.0.
Please remove mariadb und SQL Server from this PR and create a new one for 5.1 branch.

thanks

@HLeithner HLeithner added the bug label Apr 25, 2024
robertsilen added a commit to robertsilen/joomla-cms that referenced this pull request Apr 25, 2024
Considering the technical requirements (https://manual.joomla.org/docs/next/get-started/technical-requirements/) include MariaDB, I humbly suggest the README.txt should include MariaDB too. I removed "SQL Server" from list of databases, as mentioned in another discussion (joomla#43343). There is a separate PR for the .md version of the README.
@Quy
Copy link
Contributor

Quy commented Apr 25, 2024

Revert and update readme.txt.

Please remove mariadb und SQL Server from this PR and create a new one for 5.1 branch.

@robertsilen
Copy link
Contributor Author

robertsilen commented Apr 26, 2024

Revert and update readme.txt.

Please remove mariadb und SQL Server from this PR and create a new one for 5.1 branch.

I have now modified both README.txt and README.md in this PR for Joomla 4.4 to include MariaDB. For 5.1 there is this PR.

@Quy
Copy link
Contributor

Quy commented Apr 26, 2024

Please do as requested per @HLeithner in this PR.

We no longer support, SQL Server, and we support MariaDB not official in 4.x. Started with Joomla 5.0. Please remove mariadb und SQL Server from this PR and create a new one for 5.1 branch.

@Quy Quy added the Updates Requested Indicates that this pull request needs an update from the author and should not be tested. label May 21, 2024
README.md Outdated Show resolved Hide resolved
README.txt Outdated Show resolved Hide resolved
@Quy Quy removed the Updates Requested Indicates that this pull request needs an update from the author and should not be tested. label Sep 1, 2024
@Quy
Copy link
Contributor

Quy commented Sep 1, 2024

Go to https://manual.joomla.org/docs/next/get-started/technical-requirements/#requirements-for-joomla-4x to confirm only MySQL and PostgreSQL in v4.

@HLeithner HLeithner merged commit dc6aa87 into joomla:4.4-dev Sep 2, 2024
3 checks passed
@HLeithner
Copy link
Member

thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants