Skip to content
This repository has been archived by the owner on May 13, 2018. It is now read-only.

Flag für Änderungen in der Hauptsprache #6

Merged
merged 3 commits into from
Aug 11, 2013
Merged

Flag für Änderungen in der Hauptsprache #6

merged 3 commits into from
Aug 11, 2013

Conversation

jsfroth
Copy link

@jsfroth jsfroth commented Aug 8, 2013

Hi,

ich hab ein kleines Flag implementiert, das Änderungen in der Übersetzung der Hauptsprache anzuzeigen. Es soll vor allem als Hinweis für die Übersetzer der anderen Sprachen dienen, dass sie ihre Übersetzung nochmal überprüfen müssen.

Bei der Entwicklung sind 2 neue Keys hinzugekommen, die ich im Moment der Einfachheit halber hier im Code habe. Ich habe leider keine Rechte um sie im oTranCe einzupflegen.

Viele Grüße,
Josef

DSB pushed a commit that referenced this pull request Aug 11, 2013
Flag für Änderungen in der Hauptsprache
@DSB DSB merged commit b66adea into DSB:master Aug 11, 2013
@DSB
Copy link
Owner

DSB commented Aug 11, 2013

Hallo Josef,

herzlichen Dank für diese sinnvolle Erweiterung - so macht Open Soucre Spaß. ;)
Die beiden Language-Keys habe ich entsprechend im oTranCe für oTranCe angelegt.

Gruß
Daniel

@DSB
Copy link
Owner

DSB commented Aug 11, 2013

Ich habe noch einen Default value von 0 für das neue Feld hintergeschoben, da sonst die Queries beim Import fehlschlagen.
Ansonsten sieht es soweit gut aus. Als kleine Schönheitsfehler empfinde ich noch:

  • wenn das Flag anzeigt, dass eine Änderung in der Hauptsprache stattgefunden hat, man die eigene Sprache prüft und entscheidet, dass nichts geändert werden muss und speichert, so wird das Flag nicht entfernt
  • auf der Startseite sollte bei der Statistik zwischen dem grünen Haken und dem "benötigt Arbeit"-Icon eine zusätzliche Spalte "zu prüfen" eingefügt werden

Es wäre schön wenn das noch einfließen könnte. Ansonsten können wir das aber auch im weiteren Entwicklungsprozess übernehmen. Eine kurze Info - wer das übernehmen soll - wäre nicht schlecht, damit wir nicht versehentlich gleichzeitig entwickeln.

@jsfroth
Copy link
Author

jsfroth commented Aug 12, 2013

Hallo Daniel,

freut mich, dass das unsere Idee so gut ankommt :) Für uns ist es eigentlich selbstverständlich, dass wir sowas zurückfließen lassen. Immerhin profitieren wir ja auch von eurem Einsatz, warum sollten wir da nicht auch etwas beitragen?

Zu deinen Anmerkungen:
Wir entfernen das Flag an der Stelle bewusst nicht. Es kann ja sein, dass ein User Bearbeitungsrechte für mehrere Sprachen hat und nur eine bearbeitet hat. In dieser Situation wäre es falsch das Flag zu entfernen. Als Alternative haben wir ja die Möglichkeit eingebaut, das Flag durch einen Klick darauf zu entfernen. Das funktioniert in der Übersicht und im Bearbeiten-Formular.

Die Statistik ist natürlich eine gute Idee, aber leider sind wir hier bei uns in der Entwicklung im Moment etwas unterbesetzt in Blick auf unsere eigenen Projekte. Deswegen würde ich Dich bitten, dass ihr das übernehmt.

Viele Grüße,
Josef

@DSB
Copy link
Owner

DSB commented Aug 12, 2013

Hallo Josef,

danke für die Rückmeldung. Das mit der neuen Spalte in der Statistik kann ich gerne übernehmen.

Wir entfernen das Flag an der Stelle bewusst nicht. Es kann ja sein, dass ein User Bearbeitungsrechte für mehrere Sprachen hat und nur eine bearbeitet hat. In dieser Situation wäre es falsch das Flag zu entfernen.

Ja, das ist natürlich ein unschlagbares Argument an das ich selbst nicht gedacht habe. Haken dran- erledigt. ;)
Danke nochmals.

Gruß
Daniel

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

Successfully merging this pull request may close these issues.

2 participants