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

Fix dead link for ARIA alert role #17079

Merged
merged 2 commits into from
Nov 23, 2023
Merged

Fix dead link for ARIA alert role #17079

merged 2 commits into from
Nov 23, 2023

Conversation

Yordi
Copy link
Contributor

@Yordi Yordi commented Nov 23, 2023

Description & Motivation

While looking at the documentation, I ran into that link that redirected me toward a "DOCUMENT NOT FOUND". This PR is only about updating the link with the appropriate documentation.
Thanks guys, for all the work you offer us.

Additional details

NA

Related issues and pull requests

NA

Link update (previously targeting a "DOCUMENT NOT FOUND")
@Yordi Yordi requested a review from a team as a code owner November 23, 2023 10:11
@Yordi Yordi requested review from SphinxKnight and removed request for a team November 23, 2023 10:11
@github-actions github-actions bot added the l10n-fr Issues related to French content. label Nov 23, 2023
Copy link
Member

@SphinxKnight SphinxKnight left a comment

Choose a reason for hiding this comment

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

Merci pour la correction @Yordi ! une rapide suggestion quant à l'URL en HTTPS plutôt qu'en HTTP.
Vu l'état général de la page, je passerai sur une autre PR pour la refondre/la mettre à jour par rapport à son homologue anglaise.

files/fr/web/accessibility/aria/roles/alert_role/index.md Outdated Show resolved Hide resolved
@SphinxKnight SphinxKnight changed the title Update index.md Fix dead link for ARIA alert role Nov 23, 2023
Copy link
Contributor

Preview URLs

Flaws (6)

URL: /fr/docs/Web/Accessibility/ARIA/Roles/alert_role
Title: Utilisation du rôle alert
Flaw count: 6

  • broken_links:
    • Is currently http:// but can become https://
    • Can't resolve /fr/Accessibilité/ARIA/Techniques_ARIA/Utiliser_le_rôle_alertdialog
    • Can't resolve /fr/Accessibilité/ARIA/Techniques_ARIA/Utiliser_le_rôle_alertdialog
    • Can't resolve /fr/Accessibilité/ARIA/Techniques_ARIA/Utiliser_la_propriété_aria-invalid
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (0 in common of 1 possible)
External URLs (1)

URL: /fr/docs/Web/Accessibility/ARIA/Roles/alert_role
Title: Utilisation du rôle alert

@SphinxKnight SphinxKnight merged commit d0b4253 into mdn:main Nov 23, 2023
7 checks passed
@SphinxKnight SphinxKnight mentioned this pull request Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-fr Issues related to French content.
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants