[PPNL-Techteam] lidworden deployen
Bèr Kessels
ber at webschuur.com
Wed Jul 18 14:59:40 CEST 2012
Dank. Ik zag em en heb em meteen gemerged.
Je mag ook direct in de develop branch schrijven hoor, hoeft niet persé
via een pull-request; als het team klein genoeg is (twee man, moet
lukken :)) is dat geen probleem.
Ik push em vanavond even, want ik moet even zien wat er nog meer in die
release zit en meteen het releasesysteem even aanpassen zodat jullie ook
kunnen releasen.
On 18-07-12 14:47, Tycho Veltmeijer wrote:
> De postcode wordt op dit moment nog niet gevraagd in het formulier, die
> kon ik er dus niet bij doen.
> Het huisnummer wordt nu ook niet apart gevraagd en staat dus bij het
> adres (deze zouden eventueel van elkaar gescheiden kunnen worden).
Dit heb ik om verschillende redenen expres niet gedaan:
1. Postcode is geen verplichtte info voor de leden. Bovendien heeft niet
iedereen er een. Ik wilde optionele velden gewoon weglaten, wat we niet
hóeven te weten willen we ook niet in een database opslaan enzo.
2. Huisnummervelden zijn biatches. Niet iedereen heeft er een, het zijn
niet altijd nummers enzovoort. Adres heb ik daarom een vrij tekstveld
gemaakt; zo tacklen we meteen het probleem dat mensen ook een
internationaal adres kunnen opgeven enzo. KISS.
Nu we echter een ledenactie gaan doen moeten we de postcode wel hebben,
dus wil ik daarvoor een apart optioneel veld maken. Gewoon vrij
tekstveld varchar 255, zonder format-regexp-onzin want die werkt toch
nooit 100%.
> Het makkelijkste zou zijn om een extra veld aan te maken met huisnummer
> en postcode, hiervoor moet wel de database aangepast worden.
Huisnummer niet, wel voor postcode.
Ik maak me wel een beetje zorgen over newlines. Adressen bevatten
newlines, maar dat moet de secretaris maar even zelf met knippenplakken
oplossen, lijkt me. we krijgen hier wel feedback over als het niet goed
gaat.
Bèr
--
http://berk.es
Open Source Webdevelopment.
Ruby on Rails development.
Drupal Development and -consultancy.
More information about the Techteam
mailing list