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

🐛 vue-dash: Fix loader for Open Sans italic fonts #1359

Merged

Conversation

a-scolan
Copy link
Contributor

@a-scolan a-scolan commented Aug 17, 2021

Description

Correction du chargement des polices de caractères Open Sans avec Webfontloader pour intégrer les italiques nécessaires à la conformité envers la charte

Type de changement

  • Correction de bug

Checklist

  • Ma Pull Request pointe vers la bonne branche
  • Mon code suit le style de code du projet
  • J'ai effectué une review de mon propre code
  • J'ai commenté mon code, en particulier dans les parties difficiles à comprendre
  • J'ai apporté les modifications correspondantes à la documentation
  • Mes modifications ne génèrent aucun nouveau warning
  • J'ai ajouté des tests qui prouvent que mon correctif est efficace ou que ma fonctionnalité fonctionne
  • Les tests unitaires passent localement avec mes modifications
  • J'ai mis à jour le fichier Changelog

@codecov
Copy link

codecov bot commented Aug 17, 2021

Codecov Report

Merging #1359 (8a7a452) into dev (51b75ca) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               dev     #1359   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          142       142           
  Lines         1053      1053           
  Branches       147       147           
=========================================
  Hits          1053      1053           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 51b75ca...8a7a452. Read the comment docs.

@deraw deraw changed the title 🐛 vue_dash: Fix loader for Open Sans italics font 🐛 vue-dash: Fix loader for Open Sans italic fonts Aug 27, 2021
Copy link
Member

@deraw deraw left a comment

Choose a reason for hiding this comment

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

Est-ce que tu peux mettre à jour le fichier CHANGELOG.md et ajouter ceci :

- 🐛 **Corrections de bugs**
  - **template:** Correction de la police italique non chargée ([#1359](https://github.com/assurance-maladie-digital/design-system/pull/1359))

à la ligne 18 stp ?
Merci 👌

@a-scolan a-scolan force-pushed the hotfix/OpenSansItalics branch from efdbbc5 to e51c0e8 Compare August 27, 2021 10:04
@a-scolan
Copy link
Contributor Author

Est-ce que tu peux mettre à jour le fichier CHANGELOG.md et ajouter ceci :

- 🐛 **Corrections de bugs**
  - **template:** Correction de la police italique non chargée ([#1359](https://github.com/assurance-maladie-digital/design-system/pull/1359))

à la ligne 18 stp ?
Merci 👌

Fait, par contre mon commit amendé a l'air de déclencher des conflits...

@deraw
Copy link
Member

deraw commented Aug 27, 2021

Est-ce que tu peux mettre à jour le fichier CHANGELOG.md et ajouter ceci :

- 🐛 **Corrections de bugs**
  - **template:** Correction de la police italique non chargée ([#1359](https://github.com/assurance-maladie-digital/design-system/pull/1359))

à la ligne 18 stp ?
Merci 👌

Fait, par contre mon commit amendé a l'air de déclencher des conflits…

Ah oui je pense que tu dois mettre à jour ton fork avec la branche dev !

@a-scolan
Copy link
Contributor Author

Est-ce que tu peux mettre à jour le fichier CHANGELOG.md et ajouter ceci :

- 🐛 **Corrections de bugs**
  - **template:** Correction de la police italique non chargée ([#1359](https://github.com/assurance-maladie-digital/design-system/pull/1359))

à la ligne 18 stp ?
Merci 👌

Fait, par contre mon commit amendé a l'air de déclencher des conflits…

Ah oui je pense que tu dois mettre à jour ton fork avec la branche dev !

C'est le changelog qui est en conflit, mais chaque fois que votre branche va avancer mon changelog sera en conflit même si je corrige pour cette fois... Ce n'est juste pas tenable.
Il faut que l'impact changelog soit pris en compte au moment d'intégrer la pull request sur votre repo

@a-scolan a-scolan force-pushed the hotfix/OpenSansItalics branch from 7693c67 to 8a7a452 Compare September 2, 2021 14:42
@deraw deraw merged commit d5fc107 into assurance-maladie-digital:dev Sep 2, 2021
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.

3 participants