sucresware / 4sucres

Parce qu'à 2 on était pas assez.
https://4sucres.org
9 stars 4 forks source link

Ajouter l'intégration des liens imgur #57

Closed mgkprod closed 3 years ago

mgkprod commented 5 years ago

Seulement à partir d'un lien genre comme ça : https://imgur.com/JSmDfQV ça permet d'afficher autre chose que les images classiques (gifv notamment)

innocenzi commented 5 years ago

Il faut prendre en compte les deux types de liens possibles :

innocenzi commented 5 years ago

Je pense qu'il faut passer par l'API Imgur (https://apidocs.imgur.com/?version=latest#2078c7e0-c2b8-4bc8-a646-6e544b087d0f).

On peut récupérer le hash de l'image avec une expression régulière et faire une requête qui va nous retourner une URL directe (data.link) pour pouvoir l'afficher correctement.

Le problème si on ne fait pas ça, c'est que si l'utilisateur fournit un lien normal (https://imgur.com/JSmDfQV) on ne pourra pas l'afficher correctement car ce lien mène vers une page directe, et on n'a pas l'extension.

innocenzi commented 5 years ago

Ou alors utiliser https://help.imgur.com/hc/en-us/articles/211273743-Embed-Unit. Tu préfères quelle solution @mgkprod ?

innocenzi commented 5 years ago

Requiert la résolution de #75

mgkprod commented 3 years ago

Implémenté, ticket non mis à jour :(