Skip to content
Snippets Groups Projects

Feat/newsletter subscription

Merged Feat/newsletter subscription
feat/newsletter-subscription into dev
2 unresolved threads
Merged Antonin COQUET requested to merge feat/newsletter-subscription into dev
2 unresolved threads

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Hugo SUBTIL
    Hugo SUBTIL @ext.sopra.husubtil ·
    Contributor

    Remarques globales :

    Titre Description Capture
    Affichage des adresses inexistantes Potentiellement dangereux pour les attaques. Il vaut mieux mettre un message de validation du type 'Votre demande a été prise en compte'. Si l'adresse existe on la delete sinon on fait rien. image
    Inscription depuis le formulaire Ajouter une case dans le parcours d'inscription. Soit on complète l'ecran que je t'ai mis en rajoutant une case + un text pour la newsletter. Soit on rajoute carrément une étape dans le formulaire après celle-là image
    Fond blanc pour les pages Niveau accessibilité il faut qu'on est du contraste dans les champs. Je te met une capture de comment c'est fait côté formulaire image
    Idem que le premier point image
  • Hugo SUBTIL
  • Hugo SUBTIL
  • Hugo SUBTIL
    Hugo SUBTIL @ext.sopra.husubtil started a thread on an old version of the diff
    • src/app/admin/components/newsletter-users/newsletter-users.component.scss 0 → 100644
    1 .userList {
  • Hugo SUBTIL
  • Hugo SUBTIL
  • Hugo SUBTIL
  • Hugo SUBTIL
    Hugo SUBTIL @ext.sopra.husubtil ·
    Contributor

    Tes composants: newsletter-subscription.component et newsletter-unsubscription.component sont identique a 2/3 if près. Autant faire un seul composant newsletter-subscription dans lequel tu gère les deux cas. Ca fait moins de duplication de code donc plus facile a maintenir.

  • Hugo SUBTIL
    Hugo SUBTIL @ext.sopra.husubtil ·
    Contributor

    Fixer les conflits avec la dev pour pouvoir merger

  • Antonin COQUET added 1 commit

    added 1 commit

    Compare with previous version

  • Antonin COQUET added 9 commits

    added 9 commits

    Compare with previous version

  • Antonin COQUET added 2 commits

    added 2 commits

    • 17f7d5cb - add newsletter subscription on signup form
    • 9413bbe5 - Merge branch 'feat/newsletter-subscription' of...

    Compare with previous version

  • Hugo SUBTIL
    Hugo SUBTIL @ext.sopra.husubtil started a thread on an outdated change in commit 17f7d5cb
    • src/app/form/pageType.enum.ts
    23 23 structureDescription = 21,
    24 24 structureCovidInfo = 22,
    25 25 cgu = 23,
    26 newsletter = 24,
  • Antonin COQUET added 1 commit

    added 1 commit

    Compare with previous version

  • Hugo SUBTIL merged

    merged

  • Hugo SUBTIL mentioned in commit 85e1459e

    mentioned in commit 85e1459e

    • Please register or sign in to reply