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

Добавить сайт #806

Closed
Kzander3000 opened this issue Sep 12, 2024 · 12 comments
Closed

Добавить сайт #806

Kzander3000 opened this issue Sep 12, 2024 · 12 comments
Labels
Запрос на поддержку Общий тег для сайтов/клиентов/браузеров, которым нужно добавить поддержку ✅ Добавлено

Comments

@Kzander3000
Copy link

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

windows 11

Browser / Браузер

Chrome

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

Tampermonkey

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

5.2.3.

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

Добрый день.
Научите, пожалуйста, добавлять сайты самостоятельно в настройках. Я вроде использовал настройки включения Пользовательские @matches и скрипт в панели управления показывает, что включен на сайте, но видео все равно не переводит.

Меня интересуют сайты:

  1. https://learning.sap.com/
    К примеру видео: https://learning.sap.com/learning-journeys/discovering-sap-activate-implementation-tools-and-methodology/describing-sap-activate
  2. https://learninghub.sap.com/

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

No response

@ilyhalight
Copy link
Owner

ilyhalight commented Sep 12, 2024

https://learning.sap.com/

Уже поддерживается, но пока только курсы

Это тот же сайт, который ведет на learning.sap.com или на нем тоже могут быть какие-то видео?

@ilyhalight
Copy link
Owner

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

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

@Kzander3000
Copy link
Author

https://learning.sap.com/

Уже мероприятия, но пока только курсы

Это тот же сайт, который ведет на Learning.sap.com или на нем тоже могут быть какие-то видео?

Добрый день.
К сожалению для сайта https://learning.sap.com/ все равно не работает перевод, не появляется окно с переводом. Я пробовал добавить сайт и прописать логику в редактор:

{
    host: VideoService.sapLearning,
    url: "stub",
    match: /^learning\.sap\.com$/,
    selector: ".playkit-engine", - тут разные классы перечислял, в которых сидит само видео <video>...</video>
    needExtraData: true,
},

Но результата нет, возможно нужно еще что-то прописывать?
У вас код прописан следующий:

    {
        host: VideoService.sap,
        url: "https://learning.sap.com/courses/",
        match: /^learning.sap.com$/,
        selector: ".playkit-container",
        eventSelector: ".playkit-player",
        needExtraData: true,
    },

Тоже пробовал его менять, играться с классами. Не помогло((

Что касается https://learninghub.sap.com/, вы правы, там где есть видео, то страницы ссылаются на https://learning.sap.com/ соответсвенно полагаю нужно решить вопрос с этим сайтом.

@ilyhalight
Copy link
Owner

К сожалению для сайта https://learning.sap.com/ все равно не работает перевод, не появляется окно с переводом. Я пробовал добавить сайт и прописать логику в редактор:

Пока работает, только, если ссылка начинается на /courses, позже добавлю и learning-journeys

пример https://learning.sap.com/courses/opportunities-and-success-for-partners-in-sap-s-industry-cloud

@Kzander3000
Copy link
Author

Kzander3000 commented Sep 13, 2024

пример https://learning.sap.com/courses/opportunities-and-success-for-partners-in-sap-s-industry-cloud

В курсах вижу перевод в субтитрах, саму кнопку перевода, но закадрового перевода у меня нет. У вас работает?
Дайте знать, пожалуйста, когда можно будет проверить learning-journeys.

@ilyhalight
Copy link
Owner

пример https://learning.sap.com/courses/opportunities-and-success-for-partners-in-sap-s-industry-cloud

В курсах вижу перевод в субтитрах, саму кнопку перевода, но закадрового перевода у меня нет.

нужно csp удалять через tampermonkey, чтобы аудио работало

@Kzander3000
Copy link
Author

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

У меня могут быть глупые вопросы, просто я не разбираюсь в этом((

@ilyhalight
Copy link
Owner

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

У меня могут быть глупые вопросы, просто я не разбираюсь в этом((

Инструкция. Только за место "Да", нужно выбрать "Удалить"

@Kzander3000
Copy link
Author

Kzander3000 commented Sep 16, 2024

Добрый день.
Работает. Спасибо

@ilyhalight
Copy link
Owner

позже добавлю и learning-journeys

Добавил, обновите расширение

@ilyhalight ilyhalight added ✅ Добавлено Запрос на поддержку Общий тег для сайтов/клиентов/браузеров, которым нужно добавить поддержку labels Sep 16, 2024
@Kzander3000
Copy link
Author

Спасибо.
Но пока не появилось.

Правильно понимаю, что достаточно нажать кнопку Обновить тут:
2024-09-16_174602

Я попробовал еще нажать на время обновления (3d) в панели управления скрипта:
2024-09-16_174652

@ilyhalight
Copy link
Owner

Правильно понимаю, что достаточно нажать кнопку Обновить тут:

нужно нажимать "установить расширение" на странице скрипта на гитхабе и там уже предложит обновить

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Запрос на поддержку Общий тег для сайтов/клиентов/браузеров, которым нужно добавить поддержку ✅ Добавлено
Projects
None yet
Development

No branches or pull requests

2 participants