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

[4.0] Fix broken Joomla manifest and therefore broken Joomla Update check caused by recently merged PR #31504 #31525

Merged

Conversation

richard67
Copy link
Member

Pull Request for Issue # .

Summary of Changes

Fixes broken closing tag for copyright in Joomla manifest caused by recently merged PR #31504 .

Testing Instructions

Install current 4.0-dev or latest nightly, go to backend switch error reporting to "Maximum".

Then wait for the Joomla Update check to complete.

Actual result BEFORE applying this Pull Request

2020-11-29_j4-error-update-checks-xml

Expected result AFTER applying this Pull Request

No such stuff, only nice green buttons.

Documentation Changes Required

None.

@richard67
Copy link
Member Author

@nibra For your information.

@richard67
Copy link
Member Author

Could also be merged by review.

@HLeithner HLeithner merged commit 4bb3bed into joomla:4.0-dev Nov 29, 2020
@HLeithner
Copy link
Member

Thanks

@HLeithner HLeithner added this to the Joomla 4.0 milestone Nov 29, 2020
@richard67 richard67 deleted the 4.0-dev-fix-broken-joomla-manifest branch November 29, 2020 10:58
@richard67
Copy link
Member Author

Welcome ;-)

@brianteeman
Copy link
Contributor

Thanks - beat me to it

@nibra
Copy link
Member

nibra commented Nov 29, 2020

Thank you, @richard67, just checked: You found the only occurance.

@richard67
Copy link
Member Author

You found the only occurance.

@nibra You can be sure that I've checked this, too, find (Linux sh) respective findstr (Windows CMD) are my best friends.

@wilsonge
Copy link
Contributor

At least we were consistent from the merge into 3.10 #30371 :D

@richard67
Copy link
Member Author

Oh, that's so long ago, I even have forgotten about it.

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

Successfully merging this pull request may close these issues.

6 participants