FEATURE: Get notified on corpus changes (see #501) && Get notified on changes related to a topic (#152)
Content
-
Get notified on corpus changes (see #501) FEATURE : Get notified on corpus changes. Co-authored-by: Tristan Godec @TristanGdc Check Hypertopic/Porphyry#501 for more details
-
Get notified on changes related to a topic (see #152) FEATURE : Get notified on corpus changes. Co-authored-by: Tristan Godec @TristanGdc Check Hypertopic/Porphyry#152 for more details This feature has not been realized completely because the back-end part has not been fully developped. The list of the topics part has been realized for the front-end.
@benel
Merci de cette contribution @therealdarkflamemaster @TristanGdc.
Il y a deux fonctionnalités, c'est ça ? Sont-elles livrées dans deux commits différents ?
@benel Ces sont des fonctionnalité de Get notified on corpus changes #501 et Get notified on changes related to a topic #152. Dans la maquette, les buttons de cettes deux fonctionnalités sont ensemble et c'est la raison pour laquelle que l'on les faire ensemble.
Sont-elles livrées dans deux commits différents ?
En fait, non, le dernier commit est correct parce qu'il existe des codes non nécessaires dans le premier commit. Par exemple, il existait une importation de react-helmet.
Pour Get notified on changes related to a topic #152, c'est du fichier src/components/rssPage/RssTopic.jsx et quelques liens dans le fichier src/components/rssPage/Rss.jsx, comme :
<Link to="/rss/topic">
<button className="btn btn-light creationButton" >
<Trans>S'abonner à un catégorie</Trans>
</button>
</Link>
Les autres sont de Get notified on corpus changes #501
Dans la maquette, les buttons de cettes deux fonctionnalités sont ensemble et c'est la raison pour laquelle que l'on les faire ensemble.
On aurait pu faire différemment, mais OK.
En fait, non, le dernier commit est correct parce qu'il existe des codes non nécessaires dans le premier commit. Par exemple, il existait une importation de react-helmet.
Dans ce cas, il faudrait livrer votre contribution sous la forme d'un unique commit. Vous voyez comment faire ?
@benel
D'accord, j'ai trouvé git rebase pour le faire