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

ACOSS : chiffres et réalisations #299

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

Conversation

anourethpongrattana
Copy link
Contributor

modification acoss

modification acoss
update modif
update key metric defi
Copy link
Member

@AntoineAugusti AntoineAugusti left a comment

Choose a reason for hiding this comment

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

Peux-tu corriger ce fichier YAML ?

Le YAML n'est pas valide car tu as des éléments avec des guillemets doubles dans ton texte. Tu peux échapper ces guillemets doubles par exemple.

"Mes démarches de fin de contrat"

fix escaped double quotes
@AntoineAugusti
Copy link
Member

Il y a cette erreur dans la description mais également dans le title

fix escaped double quotes ++
@netlify
Copy link

netlify bot commented Nov 12, 2019

Deploy preview for entrepreneur-interet-general ready!

Built with commit c3daab2

https://deploy-preview-299--entrepreneur-interet-general.netlify.com

Copy link
Member

@AntoineAugusti AntoineAugusti left a comment

Choose a reason for hiding this comment

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

Il manque les différentes images référencées dans le fichier YAML.

@AntoineAugusti AntoineAugusti changed the title Update defis.yml ACOSS : chiffres et réalisations Nov 13, 2019
@anourethpongrattana
Copy link
Contributor Author

les images ont était ajouter via un commit sur la branche de Thierry (c'est sur que ce n'est pas la bonne pratique), car je n'arrivai pas a uploader d'images et que j'avais des bug lorsque j'essayai de cloner le projet avec ma machine du travail.

@AntoineAugusti
Copy link
Member

Peux-tu ajouter ces images sur cette PR ?

Salut @BarbeNoash ! Dans vos réalisations, il manque à chaque fois un call to action avec un lien qui redirige vers la réalisation. Pourriez-vous y remédier ? Je me dis que pour la première réalisation vous pouvez mettre le lien vers l'article de Coline, pour le deuxième et le troisième vous avez peut-être des choses sur GitHub à montrer.

L'idée de cette section étant de permettre au visiteur de comprendre tout ce que vous avez fait, c'est mieux que les réalisations soient "explorables".
@milkosofi
Copy link
Contributor

Et je reviens sur la demande d'Antoine : peux-tu remettre les photos sur cette PR pour que l'on s'y retrouve, sinon c'est trop dur de faire le suivi.

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.

4 participants