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.

Sign up for GitHub

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

Jump to bottom

Change not found video for images #24272

Closed
wants to merge 4 commits into from
Closed

Change not found video for images #24272

wants to merge 4 commits into from

Conversation

FacuBotta
Copy link
Contributor

Description

The videos were inaccessible, so I suggest replacing them with the original images. Although the images aren’t translated into Spanish, at least the instructions are preserved clearly.

Motivation

I'm glad to help!

Additional details

Related issues and pull requests

🔨 Fixes #24208

Before:

image
After:

image

@FacuBotta
Change not found video for images
6b2cb93 
The videos were inaccessible, so I suggest replacing them with the original images. Although the images aren’t translated into Spanish, at least the instructions are preserved clearly.
@FacuBotta FacuBotta requested a review from a team as a code owner October 28, 2024 19:33
@FacuBotta FacuBotta requested review from davbrito and removed request for a team October 28, 2024 19:33
@github-actions github-actions bot added the l10n-es Issues related to Spanish content. label Oct 28, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 28, 2024 •
edited
Loading

Preview URLs

Flaws (34)

URL: /es/docs/Mozilla/Add-ons/WebExtensions/Your_second_WebExtension
Title: Tu segunda extensión
Flaw count: 34

  • macros:
    • /es/docs/Mozilla/Add-ons/WebExtensions/Content_scripts does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Content_scripts
    • /es/docs/Mozilla/Add-ons/WebExtensions/Background_scripts does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Background_scripts
    • /es/docs/Mozilla/Add-ons/WebExtensions/Match_patterns does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Match_patterns
    • /es/docs/Mozilla/Add-ons/WebExtensions/Working_with_files does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Working_with_files
    • /es/docs/Mozilla/Add-ons/WebExtensions/Internationalization does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Internationalization
    • and 20 more flaws omitted
  • images:
    • Pathname should be relative to document
    • Pathname should be relative to document
  • broken_links:
    • Can't resolve /es/Add-ons/WebExtensions/Tu_primera_WebExtension
    • Can't resolve /es/Add-ons/WebExtensions/user_interface/Browser_action
    • Can't resolve /es/Add-ons/WebExtensions/manifest.json/permissions#activeTab_permission
    • Can't resolve /es/Add-ons/WebExtensions/API/tabs/insertCSS
    • Can't resolve /es/Add-ons/WebExtensions/API/tabs/removeCSS
    • and 2 more flaws omitted

(comment last updated: 2024-10-29 17:33:05)

FacuBotta and others added 3 commits October 28, 2024 20:38
@FacuBotta @github-actions
Update files/es/mozilla/add-ons/webextensions/your_second_webextensio…
fe82c71 
…n/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@FacuBotta @github-actions
Update files/es/mozilla/add-ons/webextensions/your_second_webextensio…
ba34bb2 
…n/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@FacuBotta
Merge branch 'main' into patch-2
7e2e4d9
@FacuBotta FacuBotta closed this Oct 29, 2024
@FacuBotta
Copy link
Contributor Author

Sorry, the image links aren't working because the bot removes the path to the external repository. I think it would be better to include the files directly in this repository, so they can be reliably linked from here.

@Graywolf9
Copy link
Contributor

Hi! @FacuBotta

Thank you so much for all your time and help, by now we only use the same images as the en-US version, and it is using only the image name without any full paths

@FacuBotta FacuBotta deleted the patch-2 branch December 5, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@davbrito davbrito Awaiting requested review from davbrito davbrito is a code owner automatically assigned from mdn/yari-content-es

Assignees
No one assigned
Labels
l10n-es Issues related to Spanish content.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Los enlaces a youtube no funcionan
2 participants
@FacuBotta @Graywolf9