Upgrade to Orchard Core 1.5 (OSOE-392) about open-source-orchard-core-extensions

[Piedone]

The connstring changes are necessary, so that's not an issue.

Yes, but do check if the changes on the homepage are indeed normal.

[sarahelsaig]

That problem with SQL Server connection string will also arise in the GHA.
I've already fixed that in OSOE-403, so you should merge or cherrypick from there.

[Piedone]

Great!

You'll need to publish issue-specific alphas of all projects and update the references in the NuGetTest solution right away. However, we can leave that as a last step when all the PRs are approved, so you don't need to publish them repeatedly if changes are needed.

[MZole]

I'm facing some issues with the SQL server tests, the tests can't pass the setup page, and I'm getting this error message:

This is not happening locally, I can run the tests and they pass the setup page. I have added the recommended extra stuff to our connection string (Encrypt=False; TrustServerCertificate=True) but if I remove them, I get the same error locally.

I've tried to find out what changed, and I could only find this but I'm not sure if this is relevant to us.

Also, some visual verifications fail on the VerifyHomePage test, I saw that if I delete the image, a new baseline image will be generated on the next run, I'm guessing I will be able to download that image and paste the new one in our tests folder, am I correct?

[MZole]

Thank you! I was wondering why the connection string didn't change on the tests 😁.

[MZole]

Just a quick update, this is almost done. As I understand it, I will have to open another PR for the NuGet solution update since it references our projects via NuGet, after we publish NuGet releases of the projects.

Only one test fails on Windows, the IdleTenantTests but it passes on Linux. I have checked out the failure dumps, and the log file shows that the tenant is shutting down, so this will take a bit more time for me to figure out what's causing this issue.