Skip to content

Ajout des infos de la liste Ametra#18

Merged
ptlc8 merged 4 commits into
ptlc8:masterfrom
charlyppr:Ametra
Feb 20, 2026
Merged

Ajout des infos de la liste Ametra#18
ptlc8 merged 4 commits into
ptlc8:masterfrom
charlyppr:Ametra

Conversation

@charlyppr
Copy link
Copy Markdown
Contributor

No description provided.

@ptlc8
Copy link
Copy Markdown
Owner

ptlc8 commented Feb 19, 2026

Coucou,

Content de voir votre pull request :)
Dans l'ensemble c'est good, juste quelques remarques :

  • le logo est un peu gros, il faudrait le compresser (pour passer en dessous de 400ko)
  • dans le Json il faudrait retirer les propriétés votes et écart (sinon vous serez affichés avec 0 votes) et ajouter le champs "unfinished": true sur l'année

À bientôt

@charlyppr
Copy link
Copy Markdown
Contributor Author

Ok ça marche, normalement j'ai corrigé ce que tu m'as dit ;)
Si il faut changer autres choses dit moi.

@ptlc8
Copy link
Copy Markdown
Owner

ptlc8 commented Feb 19, 2026

Nickel !
Je n'avais juste pas remarquer, mais le champ texte est invalide, il vaut FFFFFF qui n'est pas une couleur valide ; tu peux soit le mettre à #FFFFFF ou une autre couleur, ou ne pas le mettre et le texte sera noir

@charlyppr
Copy link
Copy Markdown
Contributor Author

Ah oui bien vue, c'est corrigé

@ptlc8
Copy link
Copy Markdown
Owner

ptlc8 commented Feb 19, 2026

Euh, ça n'a pas changé, aurais-tu oublié de push ?

@charlyppr
Copy link
Copy Markdown
Contributor Author

En effet, c'est bon normalement ;)

@ptlc8
Copy link
Copy Markdown
Owner

ptlc8 commented Feb 19, 2026

Parfait !

Tout est bon, je valide et je te laisse merge 🐳
cf Ambi.dev/listes quand tu auras hit le merge button

À bientôt pour les campagnes !

@charlyppr
Copy link
Copy Markdown
Contributor Author

Parfait, merci beaucoup ! Juste je n'est pas le bouton merge de mon coté

@ptlc8 ptlc8 merged commit 48a13fd into ptlc8:master Feb 20, 2026
3 checks passed
@ptlc8
Copy link
Copy Markdown
Owner

ptlc8 commented Feb 20, 2026

Oh c'est curieux, je pensais que tu pourrais

Dans tous les cas, c'est en prod !

@charlyppr
Copy link
Copy Markdown
Contributor Author

Super ! Merci à toi ;)

@charlyppr charlyppr deleted the Ametra branch February 21, 2026 20:18
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