ilyhalight / voice-over-translation

Небольшое расширение, которое добавляет закадровый перевод видео из YaBrowser в другие браузеры
MIT License
3.59k stars 237 forks source link

Udemy #781

Open mykola-merkulov opened 2 months ago

mykola-merkulov commented 2 months ago

Operating system / Операционная система

Windows 10

Browser / Браузер

Chrome

Userscript manager / Скриптовый движок

Tampermonkey

Script version / Версия скрипта

1.6.1

Describe the bug / Опишите ошибку

Здравствуйте! Согласно https://github.com/ilyhalight/voice-over-translation/wiki/%5BRU%5D-Supported-sites должна быть поддержка udemy.com/course/NAME/learn/lecture/LECTURE_ID при условии, что пользователь авторизован на сайте.

Я авторизовался на сайте, но не вижу возможности включить перевод. Что нужно сделать, чтоб плагин работал на сайте Udemy?

Спасибо!

Additional information / Дополнительная информация

No response

ilyhalight commented 2 months ago

Пока что поддерживаются, только, те курсы, которые имеют .mp4 видео в своих исходниках. Если .mp4 видео не найдено, то скрипт уведомит об этом в консоли и не будет показывать кнопку для перевода

mykola-merkulov commented 2 months ago

Спасибо за ответ! Я проверил, в курсе как раз video/mp4. Не вижу возможности включить перевод.

mykola-merkulov commented 2 months ago

Похоже изменилась структура сайта Udemy: Версия скрипта voice-over-translation: 1.6.1 Версия Chrome: 128.0.6613.85 Версия Tampermonkey: 5.2.3

Спойлер: ошибка в консоле:
userscript.html?name=%255BVOT%255D-Voice-Over-Translation.user.js&id=ba591406-d550-4871-abee-dbeab970d7a3:3626 Uncaught (in promise) VideoDataError: Failed to get video raw url for udemy
    at getVideoData (userscript.html?name=%255BVOT%255D-Voice-Over-Translation.user.js&id=ba591406-d550-4871-abee-dbeab970d7a3:3626:15)
    at async VideoHandler.getVideoData (userscript.html?name=%255BVOT%255D-Voice-Over-Translation.user.js&id=ba591406-d550-4871-abee-dbeab970d7a3:7893:7)
    at async VideoHandler.init (userscript.html?name=%255BVOT%255D-Voice-Over-Translation.user.js&id=ba591406-d550-4871-abee-dbeab970d7a3:6418:22)
ilyhalight commented 2 months ago

Похоже изменилась структура сайта Udemy: Версия скрипта voice-over-translation: 1.6.1 Версия Chrome: 128.0.6613.85 Версия Tampermonkey: 5.2.3 Спойлер: ошибка в консоле:

userscript.html?name=%255BVOT%255D-Voice-Over-Translation.user.js&id=ba591406-d550-4871-abee-dbeab970d7a3:3626 Uncaught (in promise) VideoDataError: Failed to get video raw url for udemy at getVideoData (userscript.html?name=%255BVOT%255D-Voice-Over-Translation.user.js&id=ba591406-d550-4871-abee-dbeab970d7a3:3626:15) at async VideoHandler.getVideoData (userscript.html?name=%255BVOT%255D-Voice-Over-Translation.user.js&id=ba591406-d550-4871-abee-dbeab970d7a3:7893:7) at async VideoHandler.init (userscript.html?name=%255BVOT%255D-Voice-Over-Translation.user.js&id=ba591406-d550-4871-abee-dbeab970d7a3:6418:22)

Нет, все правильно, эта ошибка как раз означает что не получилось найти ссылку на mp4 видео (на некоторых курсах апи отдает только другие форматы видео, которые не поддерживаются). Немного выше этой ошибки будет строка со списком доступных форматов Failed to find .mp4 video file in media_sources: {...}

Только что проверил, с тестовым курсом все работает

mykola-merkulov commented 2 months ago

В моем браузере установлено расширение Bulk Media Downloader, которое показывает video/mp4 на странице этого курса, по этому я и предположил, что изменилась структура сайта, и скрипт не смог распарсить ссылку на mp4 видео.

P.S. есть так же курсы, в которых Bulk Media Downloader показывать video/mp2t.

ilyhalight commented 2 months ago

Предоставьте ссылку на курс, если он бесплатный, позже гляну

Khiuk commented 1 month ago

Operating system / Операционная система Windows 10

Browser / Браузер Chrome

Userscript manager / Скриптовый движок Violentmonkey

Script version / Версия скрипта 1.7.0

Describe the bug / Опишите ошибку

На некоторых видео отсутствует кнопка перевода, предполагаю, что это вызвано другим форматов видеоролика - mp2t

Additional information / Дополнительная информация No response

dublXq commented 1 month ago

Здравствуйте. Тоже столкнулся с тем, что нет кнопки перевода. Почитал выше, причину понял. Будет ли доработка этого бага? либо другими словами расширение функционала, чтобы поддерживало не только .mp4 ?

ilyhalight commented 1 month ago

Здравствуйте. Тоже столкнулся с тем, что нет кнопки перевода. Почитал выше, причину понял. Будет ли доработка этого бага? либо другими словами расширение функционала, чтобы поддерживало не только .mp4 ?

Со временем доработаю, но не в ближайшем обновление