Skip to content

Fix/44 added more form validators when adding a new promocode

Dilana Jansen requested to merge Fix-new-promocode into dev

Omschrijving

In dit merge request heb ik formulier validatie toegevoegd en ongebruikte/foute code weggehaald:

  1. De admin krijgt nu feedback bij het invullen van het 'new promocode' formulier of het binnen de juiste eisen valt. (alleen cijfers bij 'amount', moet minimaal het getal 1 zijn etc.)
  2. Op de front-end stond een "edit" button achter elke bestaande promocode, deze functie werkte niet en is dus weggehaald.

How-to-demo

  1. Wees ingelogd met een adminemail
  2. Ga naar promocodes in de adminpanel ( http://localhost:4200/admin/promocodes )
  3. Klik op "Add new promocode"
  4. Vul de gegevens in
  5. Klik op 'Save'
  6. Nu verschijnt er een nieuwe promocode in de lijst
  7. Probeer dit met diverse foutieve opties om weer te geven welke melding er dan opkomt voor de gebruiker.

Checklist

Loop alle onderstaande punten na en zet een x in alle vakjes die van toepassing zijn.

  • Mijn pull request is voor één story/feature.
  • Elke individuele commit in dit pull request is logisch.
  • Alle code, documentatie en commits zijn in het Engels.
  • Ik heb overbodige/ongebruikte code weggegooid.
  • Mijn pull-request verwerkt geen nieuwe gevoelige informatie zonder dat ik dit heb doorgesproken met een lid van het security-team.
  • Ik heb tests toegevoegd of bijgewerkt om mijn wijzigingen te testen.
  • Als mijn wijziging veranderingen in de documentatie vereist, dan heb ik dat bijgewerkt.

Merge request reports