Ich bin mit diesem PR einverstanden.
Wie wollen wir es machnen?
- Zuerst die verbleibenden Formulare und dieser Branch kann dann ergänzt werden.
- Zuerst dieser PR, dann muss der Formular-PR angepasst werden.
Persönlich würde ich zuerst die Formulare abschließen.
"Aufnahmeantrag für natürliche Personen zur Fördermitgliedschaft" und "Aufnahmeantrag für juristische Personen zur Fördermitgliedschaft"?
Im Formular für jur. Personen würde ich "vereinbarter Betrag" durch "vereinbarte Förderung" ersetzen. Sonst ist es gut.
Also, was nach meinem Review-Chaos bleibt: Die dritte Option würde ich weglassen oder durch einen Freitext ersetzen. IMHO gilt die Klausel, dass die Förderung mit dem Vorstand vereinbart wird, grundsätzlich. Eine beliebige Summe kann man in das zweite Feld schon eintragen, dafür braucht man das dritte nicht.
- "Mit dem Vorstand wurde vereinbart" - hier sollte ein Eingabefeld sein, ohne €, weil es sich auch um andere Leistungen handeln kann
gilt das auch für natürliche Personen?
Für natürliche Personen können wir das Feld ja eigentlich weglassen. Ich hatte nicht mehr auf dem Schirm, dass wir getrennte Anträge für nat. und jur. Personen haben.
- "Ein Wechsel aus der Fördermitgliedschaft in die aktive Mitgliedschaft ist für natürliche Personen jederzeit möglich."
Baue ich ein, steht aber eigentlich auch nur auf dem Formular für natürliche Personen
Ich finde den Zusatz gerade im ersten Satz, hab das aber auch schon zweimal beim Lesen des Formulars nicht bemerkt. Wollen wir das ggf. im Titel noch unterbringen?
- Alignment der Checkboxen sieht etwas komisch aus - ich weiß aber nicht, ob sich das einfach fixen lässt
Bei mir nicht, geöffnet im Firefox.
Mein PDF-Reader macht da komische Boxen drum... das muss ich noch mal untersuchen.
- "Ich möchte € zahlen" - da fehlt ein Eingabefeld
Also bei mir ist da eins.
Ebenfalls mein PDF-Reader. Und es sind keine Punkte drunter, im Gegensatz zu den anderen. Aber die wären, soweit ich das sehe, auch deutlicher Mehraufwand.
- geboren am/in - brauchen wir das überhaupt?
IIRC war das für die Regeistrierung von Mitgliedern notwendig.
Ack.
- bei juristischen Personen könnte eine Firma interessant sein
Das ist auf dem Formular für juristische Personen der Name, oder nicht?
- Aus den letzten beiden Punkten → gibt es evtl. getrennte Formulare für Firmen und nat. Personen? (Dann kann man bei nat. Personen immer von einem Geldbetrag ausgehen und bei Firmen macht man ein Feld für die Vereinbarung)
Ah, jetzt... Ja gibt es. Sind auch beide in diesem PR drin -.- Aufnahmeantrag_foerder für natürliche Personen und Auafnahmeantrag_foerder_jur für juristische Personen.
Siehe oben - das habe ich gestern irgendwie völlig übersehen. :(
Ah, jetzt... Ja gibt es. Sind auch beide in diesem PR drin -.- Aufnahmeantrag_foerder für natürliche Personen und Auafnahmeantrag_foerder_jur für juristische Personen.
Args, Du hast natürlich Recht. War vllt schon zu spät. -.-
Ich gucke mir das unter dem Gesichtspunkt noch einmal an.
Vllt unterscheiden wir die im Titel?
Was hältst Du von einer build.sh, die das macht, was in der README als Aufruf steht?
Wir kommen in den Nitpicking-Bereich. :)
- "Ich möchte € zahlen" - da fehlt ein Eingabefeld
- "Mit dem Vorstand wurde vereinbart" - hier sollte ein Eingabefeld sein, ohne €, weil es sich auch um andere Leistungen handeln kann
- "Ein Wechsel aus der Fördermitgliedschaft in die aktive Mitgliedschaft ist für natürliche Personen jederzeit möglich."
- Alignment der Checkboxen sieht etwas komisch aus - ich weiß aber nicht, ob sich das einfach fixen lässt
- geboren am/in - brauchen wir das überhaupt?
- bei juristischen Personen könnte eine Firma interessant sein
- Aus den letzten beiden Punkten → gibt es evtl. getrennte Formulare für Firmen und nat. Personen? (Dann kann man bei nat. Personen immer von einem Geldbetrag ausgehen und bei Firmen macht man ein Feld für die Vereinbarung)
Was mir im Text noch auffällt: Da steht ein Standard-Monats-Beitrag. Wir wollten sowas anbieten, aber die Satzung sagt eigentlich aus, dass die Gegenleistung der Fördermitglieder vom Vorstand ausgehandelt wird.
D.h. man bräuchte eigenltich wenigstens Optionen, z.B.
[ ] Ich möchte 30€ / Monat zahlen
[ ] Ich möchte ___ € / Monat zahlen
[ ] Mit dem Vorstand wurde vereinbart: ___
Was sagt der Vorstand dazu?
ich hab mal einen einfachen Test im Branch experiments-cerberus gebaut. Das fühlt sich recht angenehm an. Validation von mehreren Feldern auf einmal könnte aber etwas komplexer werden.
Kannst Du daraus einen WIP-PR machen? Dann können wir da disktieren.
In diesem Stub ja. Es gibt ja noch keine Validierung.
Sobald wir eine haben, müssen da natürlich auch die passenden Testcases stehen.