Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Feature/image rights #17

Merged
merged 4 commits into from
Apr 15, 2020
Merged

Feature/image rights #17

merged 4 commits into from
Apr 15, 2020

Conversation

121593
Copy link
Contributor

@121593 121593 commented Mar 5, 2020

Permet a l'utilisateur de communiquer ses préférences pour l'utilisation de son image cf #4
@satblip stp Valide l'architecture pour les fichiers de migration de Sequelize (il y aura les seeds à coté aussi aussi)
@nicolasdb Valide ou modifie les textes de la PR stp

121593 added 4 commits March 5, 2020 14:28
@121593 121593 requested review from satblip and nicolasdb March 5, 2020 14:06
@121593 121593 mentioned this pull request Mar 5, 2020
Open
@121593 121593 linked an issue Mar 5, 2020 that may be closed by this pull request
profil de membre et photo/avatar #4
Open
@121593 121593 removed a link to an issue Mar 5, 2020
profil de membre et photo/avatar #4
Open
@nicolasdb
Copy link
Member

nicolasdb commented Mar 5, 2020 •
edited
Loading

ish,
j'ai du mal à visualiser les textes. Je les devine ici:
Dans /profil.ejs
<label class="custom-control-label" for="imageRightsConsent">J'autorise la diffusion sur openfab.be de photos/vidéos sur lesquelles j'apparais</label>

Mais hors du contexte, c'est chaud.

D'où ma petite doc pour mieux comprendre quel info doit se retrouver où, et idéalement, comment installer le minimum pour faire tourner le site en local et tester. ça sera aussi utile pour ajuster le visuel sans avoir besoin de Louis pour le push en ligne.

@nicolasdb
Copy link
Member

ok, j'ai vu que tu avais commencé le wiki ^^ cool.
merci.

satblip
satblip approved these changes Mar 5, 2020
View reviewed changes
Copy link
Member

@satblip satblip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool ça prend une chouette tournure !

@121593
Copy link
Contributor Author

121593 commented Apr 5, 2020

Pour clôturer cette PR il faut valider le texte utilisé dans le formulaire
@nicolasdb tu peux le faire ?

@nicolasdb nicolasdb mentioned this pull request Apr 14, 2020
Closed
@nicolasdb
Copy link
Member

Oui, je n'y connais rien mais ça suffira dans un premier temps.
Merci :)

@satblip
Copy link
Member

satblip commented Apr 14, 2020 •
edited
Loading

@nicolasdb tu dois approuver la PR ;) https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-request-reviews

@121593 121593 merged commit 75655c4 into openfab-lab:master Apr 15, 2020
@121593 121593 deleted the feature/image-rights branch April 15, 2020 08:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@nicolasdb nicolasdb nicolasdb approved these changes

@satblip satblip satblip approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@121593 @nicolasdb @satblip