Skip to content

[4,2] remove JDEFAULT override for com_menus#2625

Merged
heelc29 merged 2 commits intojoomlagerman:4.2-devfrom
tecpromotion:issue-2624
Oct 24, 2022
Merged

[4,2] remove JDEFAULT override for com_menus#2625
heelc29 merged 2 commits intojoomlagerman:4.2-devfrom
tecpromotion:issue-2624

Conversation

@tecpromotion
Copy link
Member

Pull Request für Issue #2624 .

Zusammenfassung der Änderungen

remove string
add string
translation

Wo wird der Sprachstring angezeigt / Wie kann getestet werden

backend

@tecpromotion tecpromotion requested a review from a team October 1, 2022 13:08
@tecpromotion tecpromotion self-assigned this Oct 1, 2022
@tecpromotion tecpromotion added this to the 4.2.4v1 milestone Oct 1, 2022
@tecpromotion tecpromotion linked an issue Oct 1, 2022 that may be closed by this pull request
@heelc29
Copy link
Member

heelc29 commented Oct 1, 2022

Bin mir nicht sicher, ob das jetzt im Core so passt. >Kommentar
image

@tecpromotion
Copy link
Member Author

@heelc29 Status dazu?

@heelc29
Copy link
Member

heelc29 commented Oct 20, 2022

@heelc29 Status dazu?

@tecpromotion Bin mir da auch ganz net sicher. Fand es davor "einheitlicher", aber mit dem Override etwas unsauber gelöst.

@tecpromotion
Copy link
Member Author

@heelc29 Status dazu?

@tecpromotion Bin mir da auch ganz net sicher. Fand es davor "einheitlicher", aber mit dem Override etwas unsauber gelöst.

Ok, sonst bitte mergen vorerst. Danke.

Copy link
Member

@heelc29 heelc29 left a comment

Choose a reason for hiding this comment

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

Danke.

@heelc29 heelc29 merged commit 6a81e95 into joomlagerman:4.2-dev Oct 24, 2022
@tecpromotion tecpromotion deleted the issue-2624 branch October 24, 2022 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

[4,2] remove JDEFAULT override for com_menus

3 participants