search

NEOSidekick / NEOSidekick.AiAssistant

1 stars 0 forks source link

FEATURE: SEO Properties Generation Backend Modules #12

Open gradinarufelix opened 3 months ago

rolandschuetz commented 3 months ago

"Anzahl der Bilder pro Seite" müsstet "Anzahl der gleichzeitig bearbeiteten Seiten" heißen. Hast du meine Vorschläge absichtlich nicht übernommen, findest du "Nur leere Fokus-Keywords" besser als "Ohne Fokus-Keyword"?

Error-Handling: Wenn es keinen einzigen Seitentyp mit AiPageBriefing Mixin gibt, dann sollten wir eine Anleitung anzeigen, dass man dieses Mixin hinzufügen muss. Bildschirmfoto 2024-04-12 um 20 15 14

Die Setting focusKeywordGeneratorModule funktionieren alle nicht

Dimensionen-Filter hast du weggelassen, weil er zu kompliziert war?

Beim absenden kommt ein Fehler: Bildschirmfoto 2024-04-12 um 20 22 20

Bildschirmfoto 2024-04-12 um 20 22 43

rolandschuetz commented 3 months ago

Wenn wir jetzt einen ganzen Backend-Bereich für uns haben und wir den Chat als immer unwichtiger sehen, lass und bitte ein Backend-Modul “Konfiguration” haben, und dort bette bitte “https://api.neosidekick.com/chat/account-context/edit” als iframe ein. Mach das gleich in dem PR.

rolandschuetz commented 3 months ago

Hier würde ich eine Exception werfen, weil es keinen Grund gibt, hier falsche Argumente zu bekommen: https://github.com/NEOSidekick/NEOSidekick.AiAssistant/blob/main/Classes/Service/AssetService.php#L82

Im Allgemeinen neigst du dazu viel zu viele Exceptions einfach zu schlucken (catch ohne Error-Handling). Damit kriegst du zwar schnell irgentwas zum laufen, aber du merkst programmierfehler seltener und debuggen wird viel schwerer.

Ich denke auch hier wollen wir einen Fehler werfen, da klappt ja ganz offensichtlich was nicht, und das muss der Nutzer wissen https://github.com/NEOSidekick/NEOSidekick.AiAssistant/blob/main/Classes/Service/AssetService.php#L94

rolandschuetz commented 3 months ago

Zeigts das bei dir in PHP-Storm nicht an?

Bildschirmfoto 2024-04-12 um 20 47 34

$nodeDatasThatNeedProcessing -> $nodesDataThatNeedProcessing (weil es ja mehrere Nodes sind)

rolandschuetz commented 3 months ago

Ich vermute das TranslationService hast du 1:1 von Neos kopiert? Dann bitte entsprechend im Code schreiben, da war doch schonmal was 😂

gradinarufelix commented 2 months ago

Wenn wir jetzt einen ganzen Backend-Bereich für uns haben und wir den Chat als immer unwichtiger sehen, lass und bitte ein Backend-Modul “Konfiguration” haben, und dort bette bitte “https://api.neosidekick.com/chat/account-context/edit” als iframe ein. Mach das gleich in dem PR.

@rolandschuetz @krsriq Wie hänge ich den API-Key an?