ilch Forum » Ilch CMS 2.x » Allgemein » [FIXED] Änderungen auf GitHub

Geschlossen
  1. #1
    User Pic
    Saarlonz Moderator
    Registriert seit
    07.08.2008
    Beiträge
    3.002
    Beitragswertungen
    235 Beitragspunkte
    Ich habe 3 Dateien auf GitHub bearbeitet aber keine Ahnung ob ich es so richtig gemacht habe, wenn mal bitte einer drüber schauen könnte.
    0 Mitglieder finden den Beitrag gut.
  2. #2
    User Pic
    corian Entwickler
    Registriert seit
    16.06.2011
    Beiträge
    841
    Beitragswertungen
    73 Beitragspunkte
    Ja habe ich schon gesehen, sieht gut aus. Pion wird das bestimmt später mergen. lächeln
    0 Mitglieder finden den Beitrag gut.
  3. #3
    User Pic
    Pion Entwickler
    Registriert seit
    25.07.2007
    Beiträge
    872
    Beitragswertungen
    51 Beitragspunkte
    Moin,

    habe es gemerged, vielen Dank.

    Optimal wäre es, wenn du einen Commit machst, den sinnig benennst und nicht viele Einzelne zum gleichen Thema.

    Ansonsten würde ich nicht sagen, dass man eine Email braucht für einen Gästebuch - Kommentar. Da wird sicherlich nicht gerne die echte Email angegeben.
    0 Mitglieder finden den Beitrag gut.
  4. #4
    User Pic
    Saarlonz Moderator
    Registriert seit
    07.08.2008
    Beiträge
    3.002
    Beitragswertungen
    235 Beitragspunkte
    Ok, das meinte ich ja damit das ich nicht weiß ob es richtig ist da ich noch nicht wirklich viel mit GitHub zu tun hatte.

    Die Änderung habe ich nur vorgenommen weil beim Eintragen E-Mail mit einem * markiert ist und mir aufgefallen ist das dies beim absenden nicht geprüft wird. Wenn das falsch war kann man es ja wieder entfernen.
    0 Mitglieder finden den Beitrag gut.
  5. #5
    User Pic
    Pion Entwickler
    Registriert seit
    25.07.2007
    Beiträge
    872
    Beitragswertungen
    51 Beitragspunkte
    Alles nice, als nächstes bitte ein ganzes Modul schreiben zwinker
    0 Mitglieder finden den Beitrag gut.
  6. #6
    User Pic
    Saarlonz Moderator
    Registriert seit
    07.08.2008
    Beiträge
    3.002
    Beitragswertungen
    235 Beitragspunkte
    Taste mich langsam ran zwinker
    0 Mitglieder finden den Beitrag gut.
Geschlossen

Zurück zu Allgemein

Optionen: Bei einer Antwort zu diesem Thema eine eMail erhalten