Szene duplizieren

Ok! Du hast es gesehen, aber die Tests laufen nicht :slight_smile:

Zögere nicht, mich um Hilfe bei den Tests zu bitten!

Danke für den Vorschlag, aber ich habe es geschafft, den Test zu reparieren, der nicht funktioniert hat. :slight_smile:
Aber ich weiß nicht warum, der Job „Docker magic“ funktioniert nicht in meinem PR. :frowning:
Hast du eine Idee?

Und jetzt habe ich einen 502-Fehler bei der Cypress-Installation :frowning:

Die Freuden des CI :slight_smile:

Die beiden Fehler ähneln temporären Fehlern, der Download-Spiegel von Cypress war wahrscheinlich zum Zeitpunkt deiner Builds down. Starte den Build neu (restart failed jobs), dann sollte es funktionieren.

Für den Docker-Magic-Build gilt dasselbe, ich denke, es liegt an einem Problem mit dem heruntergefahrenen Paket-Repository. Vielleicht war sogar Github selbst nicht verfügbar ^^

Ich kenne das Glück, wenn die CI zufällig nicht funktioniert :slight_smile:.
Es hat bei Cypress geklappt, aber nicht bei der Docker-Magie :frowning:

Ich werde einen meiner PRs aktualisieren, um zu sehen, ob es buildet.

Tatsächlich sind alle PRs gescheitert

Ich denke, ich habe die Ursache gefunden

Der Fix (cc @pierre-gilles)

Ah! Gut beobachtet, wahrscheinlich eine Änderung im Node Alpine-Image.

Ich habe deinen PR gemerged @VonOx, danke für den PR :folded_hands:

@Lokkye Du kannst master rebase!

Vielen Dank @VonOx für das Fix.
Der PR ist jetzt bereit für ein Review

Danke @Lokkye, ich habe deinen PR gesehen, das funktioniert gut bei mir und es ist viel besser mit der Auswahl des Icons :slight_smile:

Ich habe nur ein paar Rückmeldungen zu den Übersetzungen, ein paar kleine Änderungen in den Tests und die Duplizierungsfunktion des Modells, aber wirklich nichts Ernstes, es sind nur ganz kleine Änderungen!

Meine Überprüfung: Duplicate scene by callemand · Pull Request #1670 · GladysAssistant/Gladys · GitHub

Lass mich wissen, was du davon hältst, für mich kann es abgesehen davon schnell in die Produktion gehen :slight_smile:

Danke fürs Korrekturlesen. Ich habe die kleinen Korrekturen gemacht. :slight_smile:

Danke für die Korrekturen, das ist perfekt für mich!

Entschuldige, ich habe gerade noch etwas Kleines gesehen, wir versuchen wirklich, die Promise-Ketten im .then()-Modus zugunsten von async/await zu beenden. Ich habe ein .then() gefunden, das herumlag. Wenn du es einfach in await umwandelst, wäre das perfekt :grinning_face_with_smiling_eyes:

Ansonsten ist alles gut für mich, ich merge!

Edit: Wir müssen die Regel in Eslint hinzufügen, damit es automatisch funktioniert.

Edit 2: Ich habe einen PR erstellt, um die ESlint-Regel hinzuzufügen (Add promise/prefer-await-to-then eslint rule by Pierre-Gilles · Pull Request #1675 · GladysAssistant/Gladys · GitHub) :grinning_face_with_smiling_eyes:

Das ist korrigiert. Ich bevorzuge auch async/await :slight_smile:

Danke für die Fixes, das ist gut für mich!

Ich habe auf master gemerged, das wird in der nächsten Version von Gladys :slight_smile:

Das wird super praktisch sein!!
Danke @Lokkye!

Ja! Super…

Danke für diese neue Funktion

Hallo!

Ist diese Funktion bereits verfügbar?

Bald, sehr bald, spätestens mit dem nächsten Release, in ein paar Tagen wahrscheinlich… :wink:

Noch nicht, ich vermeide es, Releases durchzuführen, wenn ich danach nicht verfügbar bin, um bei Problemen die Wartung zu übernehmen, und ich habe Anfang Januar privat und beruflich sehr viel zu tun, also passte das zeitlich nicht.

Ich fliege morgen wieder nach Asien (ankomme am Freitag), nehme nächste Woche den normalen Rhythmus wieder auf und plane, dann die Version zu veröffentlichen!

Verfügbar in Gladys Assistant v4.14!