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

[5.x][Cypress][docu] HTTPS + T/S Deprecated #45085

Open
wants to merge 1 commit into
base: 5.2-dev
Choose a base branch
from

Conversation

muhme
Copy link
Contributor

@muhme muhme commented Mar 7, 2025

Summary of Changes

  1. Added note to run over HTTPS.
  2. Added troubleshooting tip if installation fails on web servers PHP version.

Testing Instructions

Read the tests/System/README.md file changes in raw and on Github.
Check if the added information is correct, useful and well formatted.

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

1. Added note to run over HTTPS.
2. Added troubleshooting tip if installation fails on web server PHP version.
@brianteeman
Copy link
Contributor

I have tested this item 🔴 unsuccessfully on e35accf


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45085.

@@ -357,3 +358,16 @@ If you encounter the following error while running the Joomla System Tests on sl
...
}
```

Copy link
Member

Choose a reason for hiding this comment

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

I think this part is not necessary as the target of Joomla is always to be compatible with the latest stable version of PHP.

Copy link
Contributor

Choose a reason for hiding this comment

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

its not just "not necessary" - I consider it to be bad advice - especially for a test suite

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