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

chore: adding licensing options #218

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

vlamy
Copy link
Contributor

@vlamy vlamy commented Nov 23, 2020

⚠️ ⚠️ PR for discussion: do not merge yet ! ⚠️ ⚠️

licence

@vlamy vlamy added the chore Side task label Nov 23, 2020
@@ -48,3 +48,6 @@ other = "Legal Information"

[allRightsReserved]
other = "Sogilis - All rights reserved - 2018"

[license]
other = "This article is pubished under the following license: "
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
other = "This article is pubished under the following license: "
other = "This article is published under the following license: "

{{ . }}
{{- else }}
CC BY-SA
{{- end }}
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Une petite indentation ne sera pas superfétatoire.
  • Plutôt que Publié le 6 novembre 2020+ Cet article est publié sous licence CC-BY, Publié le 6 novembre 2020 sous licence CC-BY ?
  • ℹ️ les - sont inutiles
  • avec default + i18n & paramètre, on peut avoir une syntaxe plus succincte : {{ i18n "license" (.Params.license | default "CC BY-SA") }} (il faut juste utiliser {{ . }} dans les traductions)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool cette syntaxe ! Je ne l'ai pas vu dans la doc d'Hugo.

Je vais modifier

Copy link
Contributor

Choose a reason for hiding this comment

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

Il manque la révision de la license.

Copy link
Contributor

@JulioJu JulioJu left a comment

Choose a reason for hiding this comment

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

On est en parfaite contradiction avec

Sogilis est propriétaire des droits de propriété intellectuelle ou détient les droits d’usage sur tous les éléments accessibles sur le site, notamment les textes, images, graphismes, logo, icônes, sons, logiciels.

Toute reproduction, représentation, modification, publication, adaptation de tout ou partie des éléments du site, quel que soit le moyen ou le procédé utilisé, est interdite, sauf autorisation écrite préalable de : Sogilis.

Toute exploitation non autorisée du site ou de l’un quelconque des éléments qu’il contient sera considérée comme constitutive d’une contrefaçon et poursuivie conformément aux dispositions des articles L.335-2 et suivants du Code de Propriété Intellectuelle.

https://sogilis.fr/mentions-legal.html

@@ -3,6 +3,7 @@ title: Un état centralisé, oui, mais sans Vuex !
author: Alban et Jean-Baptiste
date: 2020-11-06
image: /img/2020-08-31-vuex-alternative/title.jpeg
license: CC-BY
Copy link
Contributor

Choose a reason for hiding this comment

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

Il manque le numéro la révision de la license

@@ -161,10 +161,11 @@ make docker-stop

So when you open your Pull Request Netlify builds the new website and you can check the result of your Pull Request in [Netlifly](https://app.netlify.com/sites/wizardly-roentgen-e7f07e/deploys). See `deploy-preview` jobs triggered by your Pull Request to get the corresponding url.

2. Do not forget to license your work by filling the template parameter `license`, oterhwise your article will be published by default with the [CC BY-SA](https://creativecommons.org/licenses/by-sa/4.0/) creative common license.
Copy link
Contributor

Choose a reason for hiding this comment

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

Àmha, préciser CC BY-SA 4.0 dans le texte de remplacement du lien hypertexte.

{{ . }}
{{- else }}
CC BY-SA
{{- end }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Il manque la révision de la license.

@vlamy
Copy link
Contributor Author

vlamy commented Nov 23, 2020

On est en parfaite contradiction avec

Sogilis est propriétaire des droits de propriété intellectuelle ou détient les droits d’usage sur tous les éléments accessibles sur le site, notamment les textes, images, graphismes, logo, icônes, sons, logiciels.

Toute reproduction, représentation, modification, publication, adaptation de tout ou partie des éléments du site, quel que soit le moyen ou le procédé utilisé, est interdite, sauf autorisation écrite préalable de : Sogilis.

Toute exploitation non autorisée du site ou de l’un quelconque des éléments qu’il contient sera considérée comme constitutive d’une contrefaçon et poursuivie conformément aux dispositions des articles L.335-2 et suivants du Code de Propriété Intellectuelle.

https://sogilis.fr/mentions-legal.html

Tu as raison @JulioJu et pourtant je pense qu'on veut publier nos articles sous licence Creative Commons (je prends pour preuve que lorsque je lance la question, les seules personnes qui répondent sont pour ce type de licence).

Je ne pense pas que ça soit gênant de laisser les mentions légales sur le site (la surcharge de licence sur l'article est déjà un bon début), mais il y a un point à valider avec la direction de Sogilis. Comme on le suggérait sur slack, une sollication d'avis semble appropriée pour décider.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Side task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants