avatar heelc29
heelc29
2 Oct 2023

Summary of Changes

update innerHTML of #descText1 element (translation)

Testing Instructions

produce an error page are open any error html page directly
eg. domain.tl/includes/incompatible.html
switch language

Actual result BEFORE applying this Pull Request

description text is always englisch
image

Expected result AFTER applying this Pull Request

description is translated
image

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed
avatar heelc29 heelc29 - open - 2 Oct 2023
avatar richard67
richard67 - comment - 2 Oct 2023

@heelc29 Do we have this issue also in 4.4-dev, or only in 5.0-dev?

avatar heelc29
heelc29 - comment - 2 Oct 2023

Yes, but I decided to do it for J5 to avoid conflict with #41935

avatar richard67
richard67 - comment - 2 Oct 2023

But then it should be fixed in 4.4-dev, too. You could leave this PR for 5.0-dev here in addition to make it easier to resolve the conflict later on upmerge, but we definitely fix bugs in 4.4-dev and merge them up to 5 when also relevant there.

avatar HLeithner
HLeithner - comment - 2 Oct 2023

Issues which appear in 4.4 have to be fixed in 4.4 (till the release of 5.2) and will be upmerged by the maintainers to 5.x.

Could you please rebase your PR for 4.4, thanks

avatar richard67
richard67 - comment - 2 Oct 2023

@HLeithner This here is the PR, not the issue, so it should not be closed, it just needs to be rebased.

avatar HLeithner
HLeithner - comment - 2 Oct 2023

it's not possible (afaik) to rebase a joomla 5.0-dev pr to 4.4-dev branch

avatar richard67
richard67 - comment - 2 Oct 2023

it's not possible (afaik) to rebase a joomla 5.0-dev pr to 4.4-dev branch

@HLeithner Why should that not be possible?

avatar HLeithner
HLeithner - comment - 2 Oct 2023

because all commits from 5.0-dev which are not in 4.4-dev will be in this commit and you would update 4.4-dev branch to joomla 5.0. I switched this pr to 4.4 branch for demonstration.

avatar richard67
richard67 - comment - 2 Oct 2023

because all commits from 5.0-dev which are not in 4.4-dev will be in this commit and you would update 4.4-dev branch to joomla 5.0. I switched this pr to 4.4 branch for demonstration.

@HLeithner I see.

avatar HLeithner HLeithner - close - 2 Oct 2023
avatar HLeithner
HLeithner - comment - 3 Oct 2023

@heelc29 can you recreate the PR for 4.4 please?

avatar heelc29
heelc29 - comment - 3 Oct 2023

@HLeithner here we go #42030

Add a Comment

Login with GitHub to post a comment