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

Hide next meetup #132

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TristanElain
Copy link

J'ai ajouté un script node qui parse tous les meetups et le met leur statut à jour si la date est passée et s'ils ne sont pas déjà à 'done'.

J'ai une question concernant l'intégration au CI. Je pensais à lancer ce script JS pendant la phase before-script et push les éventuels fichiers modifiés par ce dernier sur la master avant de continuer l'exécution du job.
Est-ce que cette implémentation conviendrait ou bien vous aviez autre chose en tête ?

@frinyvonnick
Copy link
Member

Salut @TristanElain merci pour ta pull request 👍. L'idée de l'issue #49 était de cacher le meetup côté front (une condition qui vérifie si la date du meetup next est passée) sans intervention de notre part en attendant de venir changer la valeur dans les fichiers markdown. La solution que tu proposes nécessite de déclencher une pipeline. @charlyx avait emit l'idée de déclencher régulièrement une pipeline pour le type de changement que tu proposes. Je vais le laisser donner son avis là dessus 😄

@charlyx
Copy link
Contributor

charlyx commented Oct 24, 2019

Nous avons bien une pipeline journalière mais nous ne souhaitons pas que celle-ci modifie nos sources.
C'est pour cela que nous souhaitons mettre une condition basée sur la date lors de la génération de la page.

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

3 participants