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
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
@@ -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.


2. If you're satisfied with the result in the preview you can merge the Pull Request into `master` branch.
3. If you're satisfied with the result in the preview you can merge the Pull Request into `master` branch.

6. It's done, the deployment is automated.
4. It's done, the deployment is automated.


# Credits
1 change: 1 addition & 0 deletions site/content/posts/2020-11-06-vuex-alternative.md
Original file line number Diff line number Diff line change
@@ -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

categories:
- DÉVELOPPEMENT
tags:
3 changes: 3 additions & 0 deletions site/themes/sogilis/i18n/en.toml
Original file line number Diff line number Diff line change
@@ -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: "

3 changes: 3 additions & 0 deletions site/themes/sogilis/i18n/fr.toml
Original file line number Diff line number Diff line change
@@ -48,3 +48,6 @@ other = "Mentions Légales"

[allRightsReserved]
other = "Sogilis - Tous Droits Réservés - 2020"

[license]
other = "Cet article est publié sous licence "
7 changes: 7 additions & 0 deletions site/themes/sogilis/layouts/_default/single.html
Original file line number Diff line number Diff line change
@@ -38,6 +38,13 @@ <h1 id="articlepage-header-title-h1">{{ .Title }}</h1>
>
{{ i18n "publishdate" . }}
</time>
<br />
{{ i18n "license" }}
{{- with .Params.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.

</p>
</div>
</section>