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

FEATURE: Get notified on corpus changes (see #501) && Get notified on changes related to a topic (#152) #549

Draft
wants to merge 1 commit into
base: v7
Choose a base branch
from

Conversation

therealdarkflamemaster
Copy link

Content

  1. Get notified on corpus changes (see Get notified on corpus changes #501)
    FEATURE : Get notified on corpus changes.
    Co-authored-by: Tristan Godec @TristanGdc
    Check Get notified on corpus changes #501 for more details

  2. Get notified on changes related to a topic (see Get notified on changes related to a topic #152)
    FEATURE : Get notified on corpus changes.
    Co-authored-by: Tristan Godec @TristanGdc
    Check Get notified on changes related to a topic #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

@benel
Copy link
Member

benel commented Jun 25, 2021

Merci de cette contribution @therealdarkflamemaster @TristanGdc.

Il y a deux fonctionnalités, c'est ça ?
Sont-elles livrées dans deux commits différents ?

@therealdarkflamemaster
Copy link
Author

@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

@benel
Copy link
Member

benel commented Jun 25, 2021

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 ?

@therealdarkflamemaster therealdarkflamemaster force-pushed the ticket501_152 branch 2 times, most recently from b6eab3e to c7edf7c Compare June 25, 2021 17:33
&& Get notified on changes related to a topic (Hypertopic#152)
@therealdarkflamemaster
Copy link
Author

@benel
D'accord, j'ai trouvé git rebase pour le faire

@benel benel marked this pull request as draft April 6, 2022 13:04
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

2 participants