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: fix slack notification on dead link checker scheduled job #195

Merged
merged 1 commit into from
Oct 23, 2020

Conversation

jibidus
Copy link
Contributor

@jibidus jibidus commented Oct 20, 2020

Avant

image

Bug connu (mais non résolu, contrairement au commentaire indiqué) : act10ns/slack#114

Après

Avec Slack Notify
image

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.

degradation not degtadation.

@JulioJu
Copy link
Contributor

JulioJu commented Oct 23, 2020

Juste une note.

Vu que le fait que cela provoque une SEO degradation n'est pas certain, je n'aurais rien mis à ce sujet.

Voir ce que tu avais écrit #187 (comment) .

Enfin c'est juste un humble avis.

J'aurais juste mis .

Blog contains dead link(s). Please fix it to preserve our blog's conventions.

Inutile d'être alarmiste sachant que les impacts sur les SEO ne sont pas certains.

De plus, je le répète, sous Wikipédia les dead links présents dans les références ne sont pas supprimées. Et heureusement ! Ça noirçit les pages la question des sources dans Wikipédia, et ils n'ont pas adopté la stratégie qui a été adoptée pour le blog de Sogilis.

Et même si Wikipédia est plien de liens morts, c'est un des sites les plus populaires du monde.

De plus j'enlèverais l'icône alarmiste et mettrai à la place une icône moins angoissante (une information par exemple).

Einfin c'est juste mon très humble avis personnel. Si tu préfères ainsi (minus la faute de frappe signalée plus haut), je dirais Go ;-) !

Slack notifications with act10ns/slack@v1 on scheduled job produce ugly notification (see act10ns/slack#114)
Let's try rtCamp/action-slack-notify, which is much more customizable, in addition.
@jibidus
Copy link
Contributor Author

jibidus commented Oct 23, 2020

Typo corrigée.

Relation dead link / SEO : https://www.webrankinfo.com/dossiers/debutants/nombreuses-erreurs-404
=> semble plutôt dégrader le site cible (et non le site source)
=> #196 créé pour élucider ça une fois pour toutes

Cette PR ne modifie pas les liens morts.

L'icône alarmiste, c'est pour inciter a faire quelque chose.

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.

Cool génial :-)

@jibidus jibidus merged commit 90262b0 into master Oct 23, 2020
@jibidus jibidus deleted the fix/dead_link_scheduled_job branch October 23, 2020 15:46
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