Fixed pages showing up in wrong language #1750

JackieNim · 2021-05-09T18:41:16Z

Fix issue where links show up in the wrong language
iluwatar/java-design-patterns-web#22

sonarqubecloud · 2021-05-09T18:56:57Z

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

ohbus

@JackieNim Thank you so much for the update.

@iluwatar We may need to update the Wiki for the same changes we will be proposing to help new contributors and old ones as well to get familiarized with the new standard of adding language selection.

ohbus

Kindly take a look at the comments.

ohbus · 2021-05-11T14:20:19Z

reactor/src/main/java/com/iluwatar/reactor/app/AppClient.java

@@ -114,7 +114,7 @@ public TcpLoggingClient(String clientName, int serverPort) {

    @Override
    public void run() {
-      try (var socket = new Socket(InetAddress.getLocalHost(), serverPort)) {
+      try (Socket socket = new Socket(InetAddress.getLocalHost(), serverPort)) {


Why are we changing this?

ohbus · 2021-05-11T14:21:13Z

reactor/src/main/java/com/iluwatar/reactor/app/App.java

+        .registerChannel(tcpChannel(6666, loggingHandler))
+        .registerChannel(tcpChannel(6667, loggingHandler))
+        .registerChannel(udpChannel(6668, loggingHandler))


Please explain why are you reverting this change?

#1709 was used to patch the changes. Please show relevant necessity.

ohbus · 2021-05-11T14:21:35Z

reactor/src/main/java/com/iluwatar/reactor/app/AppClient.java

+    service.execute(new TcpLoggingClient("Client 1", 6666));
+    service.execute(new TcpLoggingClient("Client 2", 6667));
+    service.execute(new UdpLoggingClient("Client 3", 6668));
+    service.execute(new UdpLoggingClient("Client 4", 6668));


Please do let me know the reason for this change.

These have been introduced at #1709 by myself.

ohbus · 2021-05-11T14:31:56Z

version-number/README.md

 categories:
 - Concurrency


There might be an issue with categories of version-number pattern. Please check L8

Suggested change

categories:

- Concurrency

categories: Concurrency

ohbus · 2021-05-11T15:19:08Z

ko/adapter/README.md

@@ -2,8 +2,9 @@
 layout: pattern
 title: Adapter
 folder: adapter
-permalink: "/patterns/adapter/"
+permalink: "/patterns/adapter/ko"


This detail must have slipped when we approved the Korean Language translation PR but there seems to be an additional double quotation in all permalinks.

Need to check if this should be like how it is or not. Hey, @iluwatar any thoughts on this?

Suggested change

permalink: "/patterns/adapter/ko"

permalink: /patterns/adapter/ko

ohbus · 2021-05-11T15:33:32Z

ko/singleton/README.md

@@ -2,8 +2,9 @@
 layout: pattern
 title: Singleton
 folder: singleton
-permalink: "/patterns/singleton/"
+permalink: "/patterns/singleton/ko"


Korean pattern readme for permalink is not unified like other patterns.

Suggested change

permalink: "/patterns/singleton/ko"

permalink: /patterns/singleton/ko

ohbus · 2021-05-11T15:33:44Z

ko/strategy/README.md

@@ -2,8 +2,9 @@
 layout: pattern
 title: Strategy
 folder: strategy
-permalink: /patterns/strategy/
+permalink: "/patterns/strategy/ko"


Korean pattern readme for permalink is not unified like other patterns.

Suggested change

permalink: "/patterns/strategy/ko"

permalink: /patterns/strategy/ko

ohbus · 2021-05-11T15:43:46Z

fr/README.md

+    afin qu'on puisse avoir un affichage agréable comme sur un 
    site web par exemple -->

 # Les patrons de conception implémentés en Java

 ![Java CI with Maven](https://github.com/iluwatar/java-design-patterns/workflows/Java%20CI%20with%20Maven/badge.svg)
 [![License MIT](https://img.shields.io/badge/license-MIT-blue.svg)](https://raw.githubusercontent.com/iluwatar/java-design-patterns/master/LICENSE.md)
-[![Lines of Code](https://sonarcloud.io/api/project_badges/measure?project=iluwatar_java-design-patterns&metric=ncloc)](https://sonarcloud.io/dashboard?id=iluwatar_java-design-patterns)
+[![Lines of Code](https://sonarcloud.io/api/project_badges/measure?project=iluwatar_java-design-patterns&metric=ncloc)](https://sonarcloud.io/dashboard?id=iluwatar_java-design-patterns) 
 [![Coverage](https://sonarcloud.io/api/project_badges/measure?project=iluwatar_java-design-patterns&metric=coverage)](https://sonarcloud.io/dashboard?id=iluwatar_java-design-patterns)
 [![Join the chat at https://gitter.im/iluwatar/java-design-patterns](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/iluwatar/java-design-patterns?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 <!-- ALL-CONTRIBUTORS-BADGE:START - Do not remove or modify this section -->


Why are you making this change?

ohbus · 2021-05-11T15:44:30Z

fr/README.md

 # Introduction

-Les patrons de conception (design patterns) sont les meilleures 
-pratiques formalisées qu’un programmeur peut utiliser pour résoudre
-des problèmes courants lors de la conception d’une application 
-ou d’un système.
+Le patron de conception est la meilleure formatalisation qu'un programmeur
+peut utiliser pour résoudre un problème lors d'une conception d'une application/system.

-Les patrons de conception peuvent accélérer le processus de
+Le patron de conception (design pattern) est là pour accélérer le processus de 
 développement en fournissant des paradigmes éprouvés.

-La réutilisation de patrons de conception aide à se prémunir des problèmes subtils
-qui causent des problèmes majeurs, et cette pratique augmente la lisibilité
-du code pour les développeurs et architectes familiers de ces concepts.
+La réutilisation de patron de conception aide à prévenir des problèmes subtiles mais
+qui sont à l'origine de problèmes majeures, comme cette pratique augmente la lisibilitée
+du code par les développeurs/architectes familiers avec l'utilisation de ces concepts.

 # Commencer


These translation have been improved in #1730

Why are you trying to revert these changes?

ohbus · 2021-05-11T15:46:46Z

fr/README.md

+Ce site présente des modèles de conception Java. Les solutions ont été développées par des
+développeurs et architectes expérimentés de la communauté open source. Les modèles peuvent être parcourus par leurs descriptions de haut niveau ou en regardant leur code source. Les exemples de code source sont bien commentés et peuvent être considérés comme tutoriels de programmation sur la façon d'implémenter un modèle spécifique. Nous utilisons le plus technologies Java open source éprouvées au combat.

-Avant de vous plonger dans le contenu, vous devriez vous familiariser avec divers
+Avant de plonger dans le matériau, vous devez vous familiariser avec divers
 [Principes de conception de logiciels](https://java-design-patterns.com/principles/).

-Tous les modèles doivent être aussi simples que possible.
-Vous devriez commencer par les principes KISS, YAGNI et 
-Faites La Chose La Plus Simple Qui Fonctionne.
-La complexité et les modèles ne devraient être introduits
-que lorsqu’ils sont nécessaires pour une extensibilité pratique.
+Toutes les conceptions doivent être aussi simples que possible. Vous devriez commencer par KISS, YAGNI,
+et faire la chose la plus simple qui pourrait éventuellement fonctionner principes. Complexité et
+les modèles ne devraient être introduits que lorsqu'ils sont nécessaires pour
+extensibilité.

-Une fois que vous êtes familiarisé avec ces concepts, vous pouvez commencer à explorer les
-[modèles de conception disponibles](https://java-design-patterns.com/patterns/)
-par n’importe laquelle les approches suivantes&nbsp:
+Une fois que vous êtes familiarisé avec ces concepts, vous pouvez commencer à explorer
+[modèles de conception disponibles](https://java-design-patterns.com/patterns/) par tout
+des approches suivantes

- Recherchez un modèle spécifique par son nom.
-  Vous n’en trouvez pas&nbsp;? Veuillez signaler un nouveau modèle [ici](https://github.com/iluwatar/java-design-patterns/issues).
- Utilisation de balises telles que `Performance`, `Gang of Four` ou `Data access`.
- Utilisation des catégories de modèles, `Creational`, `Behavioral` et autres.
+ - Recherchez un modèle spécifique par son nom. Vous n'en trouvez pas? Veuillez signaler un nouveau modèle [ici](https://github.com/iluwatar/java-design-patterns/issues).
+ - Utilisation de balises telles que `Performance`, `Gang of Four` ou `Data access`.
+ - Utilisation des catégories de modèles, `Créatif`, `Comportemental` et autres.

-Nous espérons que vous trouverez les solutions orientées objet présentées
-sur ce site utiles dans vos architectures et que vous aurez autant 
-de plaisir à les apprendre que nous en avons eu à les développer.
+J'espère que les solutions orientées objet présentées sur ce site vous seront utiles
+dans vos architectures et ayez autant de plaisir à les apprendre que nous en avons eu à les développer.

 # Comment contribuer

 Si vous souhaitez contribuer au projet, vous trouverez les informations pertinentes dans
-notre [wiki développeur](https://github.com/iluwatar/java-design-patterns/wiki).
-Nous vous aiderons répondrons à vos questions dans le [Gitter chatroom](https://gitter.im/iluwatar/java-design-patterns).
+notre [wiki développeur](https://github.com/iluwatar/java-design-patterns/wiki). Nous aiderons
+vous et répondez à vos questions dans le [Gitter chatroom](https://gitter.im/iluwatar/java-design-patterns).

 # Licence

-Ce projet est concédé sous les termes de la licence MIT.
+Ce projet est concédé sous les termes de la licence MIT.


These translations have been improved in #1730

Please explain with an appropriate reason as to why are you trying to make some changes that revert certain commits where improvements have been made.

I think there was a problem when I tired forking the project 🤔 Let me try again

@ohbus #1752 is the new PR

iluwatar · 2021-05-14T10:52:59Z

@iluwatar We may need to update the Wiki for the same changes we will be proposing to help new contributors and old ones as well to get familiarized with the new standard of adding language selection.

Good point @ohbus. I created a new issue #1755 about this.

ohbus · 2021-05-14T15:28:33Z

@iluwatar Shall we reopen #1752 against this and re-review the whole doc again?

Or is there any way that you know where @JackieNim can rrecoverthis deleted fork to make this PR open again?

iluwatar · 2021-05-14T15:34:51Z

Since the repository has been deleted I don't know any other way than reopening #1752

Fixed pages showing up in wrong language

23760fc

JackieNim mentioned this pull request May 9, 2021

Some patterns displayed in wrong language iluwatar/java-design-patterns-web#22

Closed

ohbus self-assigned this May 11, 2021

ohbus added the status: under review label May 11, 2021

ohbus reviewed May 11, 2021

View reviewed changes

ohbus suggested changes May 11, 2021

View reviewed changes

JackieNim mentioned this pull request May 11, 2021

Fixed pages showing up in wrong language #1752

Merged

JackieNim closed this May 11, 2021

iluwatar added this to the 1.25.0 milestone May 14, 2021

	permalink: "/patterns/adapter/ko"
	permalink: /patterns/adapter/ko

	permalink: "/patterns/singleton/ko"
	permalink: /patterns/singleton/ko

	permalink: "/patterns/strategy/ko"
	permalink: /patterns/strategy/ko

Uh oh!

Fixed pages showing up in wrong language #1750

Fixed pages showing up in wrong language #1750

Uh oh!

Conversation

JackieNim commented May 9, 2021

Uh oh!

sonarqubecloud bot commented May 9, 2021

Uh oh!

ohbus left a comment

Choose a reason for hiding this comment

Uh oh!

ohbus left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

iluwatar commented May 14, 2021

Uh oh!

ohbus commented May 14, 2021

Uh oh!

iluwatar commented May 14, 2021

Uh oh!

Uh oh!