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.

Sign up for GitHub

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

Jump to bottom

Sélectionne le modèle de métadonnées dans le panneau latéral #280

Closed
wants to merge 10 commits into from
Closed

Sélectionne le modèle de métadonnées dans le panneau latéral #280

wants to merge 10 commits into from

Conversation

thom4parisot
Copy link
Member

@thom4parisot thom4parisot commented Dec 2, 2020 •
edited
Loading

image

closes #147
closes #146
closes #157

@thom4parisot thom4parisot marked this pull request as draft December 2, 2020 10:52
@ggrossetie ggrossetie mentioned this pull request Dec 2, 2020
Merged
thom4parisot
thom4parisot commented Dec 2, 2020
View reviewed changes
front/gatsby/src/schemas/default/ui-schema-editor.json Outdated
"publisher": {
"ui:title": "Publisher"
},
"journalsubtitle": {
Copy link
Member Author

Choose a reason for hiding this comment

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

@antoinentl @marviro cf. #157, est-ce que journalsubtitle doit également apparaitre dans le schéma de métadonnées de Sens Public ?

@ggrossetie ggrossetie force-pushed the issue-269-metadata-form-edit branch from 0adefff to 29521d4 Compare December 17, 2020 15:03
Base automatically changed from issue-269-metadata-form-edit to master December 17, 2020 15:12
@thom4parisot thom4parisot force-pushed the issue-192-multiple-metadata branch 5 times, most recently from f45e5fa to 5fd3739 Compare January 13, 2021 15:47
@thom4parisot
Copy link
Member Author

thom4parisot commented Jan 26, 2021 •
edited
Loading

Remarques lors de l'appel du jour :

Pour la transformation des métadonnées lors des exports, suivre #311.

@thom4parisot thom4parisot mentioned this pull request Feb 12, 2021
Closed
5 tasks
@thom4parisot
Copy link
Member Author

@antoinentl un feedback utile pour terminer cette PR serait de clarifier le comportement attendu :

  • sélection du modèle lors de la création de l'article, ou dans l'article ?
  • l'apparence/emplacement du sélecteur de modèle (à cet endroit, ailleurs ?)
  • que faire si le modèle vers lequel on bascule contient un même identifiant de champ (exemple : keywords), mais est de nature différente (liste d'objets vs. liste de chaines de caractère) ?

@thom4parisot thom4parisot force-pushed the issue-192-multiple-metadata branch 2 times, most recently from 09a46e8 to a1f160a Compare March 4, 2021 07:27
@thom4parisot thom4parisot force-pushed the issue-192-multiple-metadata branch from a1f160a to 32a208a Compare March 4, 2021 07:32
@thom4parisot
Copy link
Member Author

Generic

image

Sens public

image

@thom4parisot thom4parisot marked this pull request as ready for review March 4, 2021 20:49
@thom4parisot
Copy link
Member Author

Je clos la proposition, la proposition a 1 an et le sujet n'est jamais revenu dans les conversations.

On réutilisera ce travail si ce besoin était de nouveau à prioriser.

@antoinentl
Copy link
Member

Je me permets de rouvrir, après les travaux en cours ça va devenir une priorité. Je documenterai ici les résultats de nos échanges en novembre ou décembre 2021.

@antoinentl antoinentl reopened this Nov 10, 2021
@thom4parisot
Copy link
Member Author

thom4parisot commented Nov 10, 2021 •
edited
Loading

Entendu — le code a bougé en 1 an et repartir de cette pull request nous prendra plus de temps que de recommencer de zéro. Je ferme cette PR et j'ouvre une issue qui consigne le besoin, et le lien avec ce travail déjà réalisé.

@thom4parisot thom4parisot mentioned this pull request Nov 10, 2021
Open
@thom4parisot thom4parisot deleted the issue-192-multiple-metadata branch November 10, 2021 21:12
@antoinentl
Copy link
Member

Merci @oncletom 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antoinentl antoinentl Awaiting requested review from antoinentl

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@thom4parisot @antoinentl @ggrossetie