Überarbeitung demo_addon #55
Conversation
Erster Stand der Überarbeitung/Erweiterung des demo_addons
pages/config.rex_config_form.php
Outdated
| $field = $form->addLinklistField('articles'); | ||
| $field->setLabel($this->i18n('config_articles')); | ||
|
|
||
| // 2.5 Kategorienauswahl |
There was a problem hiding this comment.
Kann ich nix dazu sagen. Am besten mal nen Screenshot wie es aussieht und wie du es gerne hättest posten..
Ja, das geht. :) |
|
Holy moly, @aeberhard, danke dafür! 🙌 |
CHANGELOG.md
Outdated
| @@ -1,5 +1,21 @@ | |||
| # Changelog | |||
|
|
|||
| ## Version 1.2.0beta1 // 31.01.2020 | |||
There was a problem hiding this comment.
Kleine Anmerkung: Changelog und Versionserhöhung besser nicht innerhalb eines PRs anbieten. Sowas macht man lieber ganz am Ende direkt im Master-Branch, wenn klar ist, welche Anpassungen von allen Beteiligten in ein Release geflossen sind.
There was a problem hiding this comment.
Ja ist mir bekannt. Ich würde ja auch die 1.2.0 als angestrebtes Ende bezeichnen.
Daher und auch um zu zeigen was der Stand ist hier bereits die Changelog
assets/js/script.js
Outdated
| @@ -8,7 +13,7 @@ | |||
| // https://api.jquery.com/ready/ | |||
| $(document).ready(function () { | |||
There was a problem hiding this comment.
Hier sollten wir vielleicht besser rex:ready verwenden: https://redaxo.org/doku/master/addon-assets#rexready
There was a problem hiding this comment.
Da hast Du vollkommen recht! Wird gemacht! Schon gut solche Reviews :)
thx dirk Co-Authored-By: Dirk Schürjohann <ds@decaf.de>
- Demo-Tabelle und Testdaten in install.php - Beispiel Tabellenverwaltung mit rex_list/rex_form - Textänderungen - Beispiele für Zugriff auf Addon-Properties - Ordner lib und Klasse demo_addon_logger hinzugefügt
- diverse Textänderungen - Prüfung Geburtsdatum CustomFunction demoAddon_checkBirthdate - .php_cs.dist and REDAXO angepasst - lib/demo_addon_rex_form.php hinzugefügt - Tabellenverwaltung angepasst
|
Sehr umfangreicher PR. Den wird vermutlich niemand so einfach mehr reviewen können und wollen. Wenn die Anpassungen für dich okay sind, @aeberhard, würde ich ja vorschlagen, dass du gerne mergest. |
Co-Authored-By: Markus Staab <maggus.staab@googlemail.com>
Co-Authored-By: Markus Staab <maggus.staab@googlemail.com>
|
Jo @schuer ist etwas umfangreicher geworden :) Geplant ist hier auch auch möglichst viele EP's und die Verarbeitung der Parameter als Beispiele zu integrieren. Nichtsdestotrotz hat ja vielleicht auch jemand Zeit draufzuschauen ^^ |
|
Also ich nehme mich da raus, traue es mir nicht zu und vertraue da ganz auf euch. |
|
LGTM 👍 |
Verwaltung Demo-Tabelle und Textänderungen - Mindestversion PHP 7.3 in package.yml und Changelog - Demo-Einstellungen, beide Versionen angepasst - .php_cs.dist angepasst - diverse Textänderungen
Tabellenverwaltung, Extensionpoints usw
Erster Stand der Überarbeitung/Erweiterung des demo_addons
@staabm @gharlan @bloep @schuer @alexplusde
Servus Jungens. Habe mal das demo_addon initial überarbeitet.
Mithilfe Anregungen und Unterstützung bei Texten, Übersetzungen usw. sehr willkommen :)
In der DESCRIPTION.de.md steht auch noch einiges drin was geplant ist ...
Kann man das Addon aus einem PR downloaden um sich das auch live anzuschauen?
Ansonsten würde ich noch ScreenShots machen ...