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

Améliorer la gestion des moyens de contact entre usagers et les producteurs de donnée #150

Merged
merged 12 commits into from
Jul 5, 2016

Conversation

ThibautGery
Copy link
Contributor

@ThibautGery ThibautGery commented Jun 30, 2016

fix #149

Ajout de l'adresse mail ou d'un lien vers un formulaire de contact dans les fiches d'API

meta-api

Mise en place d'une page de contact

contact-page

Template de création d'API

contact: #moyen de contact, soit par mail, soit par formulaire de contact
  email: contact@ma-super-api.fr #adresse mail
  link: ma-super-api.fr/contact #lien vers le formulaire de contact

@@ -5,6 +5,8 @@ doc_tech: https://api.taxi/documentation
access_link: http://le.taxi/join.html
domain: https://le.taxi
contract: OUVERT sous contrat
contact:
link: http://le.taxi/join.html
Copy link
Member

Choose a reason for hiding this comment

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

httpS

@MattiSG
Copy link
Member

MattiSG commented Jul 1, 2016

Il me semblerait pertinent de forcer la présence d'un lien de contact par validation à la CI.

@ThibautGery
Copy link
Contributor Author

Oui mais c'est pour l'instant pas possible car l'API travaux de Rennes n'a pas de formulaire de contact. Je viens de le demander. Une fois qu'on a tout les liens, je ferais une nouvelle PR.

{% if page.contact contains "@" %}
<a href="mailto:{{ page.contact }}?subject={{ "[API gouv] Contact" | url_encode }}" target="_blank">{{ page.contact }}</a>
{% else %}
<a href="{{ page.contact }}" target="_blank">Formulaire</a>
Copy link
Member

Choose a reason for hiding this comment

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

Pourquoi target="_blank" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@MattiSG la différence entre le site api.gouv.fr et les sites des API étaient souvent mal compris : les gens ne comprenaient pas qu'ils étaient sortis d'api.gouv donc j'ai mis les ouvertures de lien extérieur dans de nouveaux onglets

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'est bizarre, il met met outdated diffIl n'y a pas de commit près :'(

Copy link
Member

Choose a reason for hiding this comment

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

Mmh. Ok. Ajoute un rel="noopener", a minima. Ça ne protègera que 45% des gens pour le moment, mais on peut espérer que ça augmentera d'ici à ce qu'une attaque nous vise…

@MattiSG
Copy link
Member

MattiSG commented Jul 4, 2016

Mmh. rel="noopener" is standard according to WhatWG.

@ThibautGery
Copy link
Contributor Author

J'ai ouvert une PR pour mettre à jour html5validator svenkreiss/html5validator#19

@ThibautGery
Copy link
Contributor Author

PR merged

{% if page.contact contains "@" %}
<a href="mailto:{{ page.contact }}?subject={{ "[API gouv] Contact" | url_encode }}" target="_blank">{{ page.contact }}</a>
{% else %}
<a href="{{ page.contact }}" target="_blank">Formulaire</a>
Copy link
Member

Choose a reason for hiding this comment

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

Tu as ajouté rel="noopener" partout sauf sur le lien qui a généré la discussion sur le sujet 😂

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😭 😭 😭 😭 😭 😭 😭

@MattiSG
Copy link
Member

MattiSG commented Jul 5, 2016

GTM

@ThibautGery ThibautGery merged commit 13c54e8 into gh-pages Jul 5, 2016
@ThibautGery ThibautGery deleted the update-contact branch July 5, 2016 16:02
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.

2 participants