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

Excerpt #42

Closed
wants to merge 7 commits into
base: master
from

Conversation

2 participants
@B4rb3rouss
Contributor

B4rb3rouss commented Dec 31, 2017

attemps to fix #314

@B4rb3rouss B4rb3rouss referenced this pull request Dec 31, 2017

Open

Excerpt of Blogposts #314

@remrem

This comment has been minimized.

Show comment
Hide comment
@remrem

remrem Jan 30, 2018

Contributor

Pourquoi ne pas avoir modifier directement l'addon "read more" ? Le nom me semble pertinent non ?

Contributor

remrem commented Jan 30, 2018

Pourquoi ne pas avoir modifier directement l'addon "read more" ? Le nom me semble pertinent non ?

@B4rb3rouss

This comment has been minimized.

Show comment
Hide comment
@B4rb3rouss

B4rb3rouss Jan 30, 2018

Contributor

readmore faisait tout autre chose à la fin d'un article, c'est pourquoi je n'ai pas changé.
Il faudrait dans ce cas déplacer le code de "excerpt" dans "readmore", mais je n'ai pas osé n'étant pas l'auteur de readmore.

Contributor

B4rb3rouss commented Jan 30, 2018

readmore faisait tout autre chose à la fin d'un article, c'est pourquoi je n'ai pas changé.
Il faudrait dans ce cas déplacer le code de "excerpt" dans "readmore", mais je n'ai pas osé n'étant pas l'auteur de readmore.

@remrem

This comment has been minimized.

Show comment
Hide comment
@remrem

remrem Jan 30, 2018

Contributor

cf. #35 , en fait, le but de la manœuvre est de modifier :

  • readmore, qui doit afficher une "introduction" (comme ton addon) sur l'index et les pages suivantes et précédentes de la liste des articles.
  • relatedposts, qui doit afficher X articles relatif à un article en fin d'un article
    ;)
Contributor

remrem commented Jan 30, 2018

cf. #35 , en fait, le but de la manœuvre est de modifier :

  • readmore, qui doit afficher une "introduction" (comme ton addon) sur l'index et les pages suivantes et précédentes de la liste des articles.
  • relatedposts, qui doit afficher X articles relatif à un article en fin d'un article
    ;)
@B4rb3rouss

This comment has been minimized.

Show comment
Hide comment
@B4rb3rouss

B4rb3rouss Jan 30, 2018

Contributor

Il suffit de renommer les fichiers/dossiers alors?

Contributor

B4rb3rouss commented Jan 30, 2018

Il suffit de renommer les fichiers/dossiers alors?

@remrem

This comment has been minimized.

Show comment
Hide comment
@remrem

remrem Jan 30, 2018

Contributor

pour excerpt, yep mais, voir le nommage des fonctions, variables ....

Contributor

remrem commented Jan 30, 2018

pour excerpt, yep mais, voir le nommage des fonctions, variables ....

@B4rb3rouss

This comment has been minimized.

Show comment
Hide comment
@B4rb3rouss

B4rb3rouss Jan 31, 2018

Contributor

ok.
Je pousse dans dev ou master?

Contributor

B4rb3rouss commented Jan 31, 2018

ok.
Je pousse dans dev ou master?

@B4rb3rouss B4rb3rouss referenced this pull request Jan 31, 2018

Merged

Readmore #49

@B4rb3rouss

This comment has been minimized.

Show comment
Hide comment
@B4rb3rouss

B4rb3rouss Jan 31, 2018

Contributor

J'ai fait une nouvelle PR pour que ça soit propre : #49

Contributor

B4rb3rouss commented Jan 31, 2018

J'ai fait une nouvelle PR pour que ça soit propre : #49

@B4rb3rouss B4rb3rouss closed this Jan 31, 2018

@remrem

This comment has been minimized.

Show comment
Hide comment
@remrem

remrem Jan 31, 2018

Contributor

Nickel, j'essaye de valider tout ça dans la journée.

Contributor

remrem commented Jan 31, 2018

Nickel, j'essaye de valider tout ça dans la journée.

@B4rb3rouss

This comment has been minimized.

Show comment
Hide comment
@B4rb3rouss

B4rb3rouss Jan 31, 2018

Contributor

Super :)
Ça avance ::)

Contributor

B4rb3rouss commented Jan 31, 2018

Super :)
Ça avance ::)

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