Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajout du modèle Mandataire #41

Merged
merged 13 commits into from
Oct 27, 2022
Merged

Ajout du modèle Mandataire #41

merged 13 commits into from
Oct 27, 2022

Conversation

tmerlier
Copy link
Contributor

Contexte

Cette PR prépare la migration du fichier client.yml sur mongodb en ajoutant un modèle Mandataire qui sera lié au prochain modèle Client.

Mandataire

{
  _id: '7n51dae287088f21afd07c580',
  nom: 'Nom du mandataire',
  email: 'mandataire@email.fr',
  _createdAt: '1970-01-01',
  _updatedAt: '1970-01-01',
}

Routes

Toutes les routes nécéssitent une authentification administrateur

/mandataires

  • Créer un mandataire : POST /mandataires/
  • Récupérer un mandataire : GET /mandataires/:mandataireId
  • Modifier un mandataire : PUT /mandataires/:mandataireId

Évolutions nécessaires à la mise en place du nouveau modèle

  • Ajout de Joi aux dépendances
  • Mise en place d'un middleware ensureIsAdmin qui s'assure de la présence du jeton ADMIN_TOKEN autorisant l'action demandée
  • Ajout de la variable ADMIN_TOKEN au fichier .env.sample

Cette PR est issue d'un découpage de #39 afin de faciliter la revue et l'intégration.

@tmerlier tmerlier marked this pull request as ready for review October 24, 2022 12:30
@tmerlier tmerlier self-assigned this Oct 24, 2022
lib/mandataires/routes.js Outdated Show resolved Hide resolved
lib/mandataires/__tests__/routes.js Outdated Show resolved Hide resolved
.env.sample Outdated Show resolved Hide resolved
lib/mandataires/model.js Show resolved Hide resolved
lib/mandataires/model.js Outdated Show resolved Hide resolved
lib/mandataires/model.js Outdated Show resolved Hide resolved
lib/mandataires/model.js Outdated Show resolved Hide resolved
lib/mandataires/model.js Outdated Show resolved Hide resolved
@tmerlier tmerlier merged commit 9238b9f into master Oct 27, 2022
@tmerlier tmerlier deleted the tmerlier/mandataires branch October 27, 2022 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants