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

Асинхронная обработка перевода и получения субтитров #694

Merged
merged 3 commits into from
Jul 24, 2024

Conversation

SashaXser
Copy link

В процессе работы возникли некоторые трудности с кодом Даниила, что потребовало внесения дополнительных изменений.
Была идея отказаться от использования различных типов сборки и интегрировать функции версии Cloudflare в основную версию. Проксирование через Cloudflare планировалось включать с помощью переменной в базе данных, которая автоматически задаётся при ошибках или неподдерживаемых расширениях. Однако я решил отложить это на потом.

В процессе работы возникли некоторые трудности с кодом Даниила, что потребовало внесения дополнительных изменений. В дальнейшем планируется отказаться от текущих типов сборки и внедрить новую переменную в БД для проксирования запросов через Cloudflare. Это решение обеспечит более простую и эффективную обработку запросов, а также возможность автоматической настройки в случае ошибок.

На данном этапе реализована только асинхронность для функций перевода и получения субтитров. В будущем будет проведена доработка для полной интеграции новой переменной проксирования.
@ilyhalight
Copy link
Owner

Можешь адаптировать под эту ветку? Там, уже сделан перенос с каллбеков и немного изменена логика работы, но для релиза еще не все сделано и протестировано

@SashaXser
Copy link
Author

Можешь адаптировать под эту ветку? Там, уже сделан перенос с каллбеков и немного изменена логика работы, но для релиза еще не все сделано и протестировано

Завтра попробую

@SashaXser
Copy link
Author

SashaXser commented Jul 10, 2024

Вот список найденных багов (будет дополняться):

  1. Окно выбора субтитров перекрывается в полноэкранном режиме (например, в плеере Kodik).
  2. При передвижении субтитров в полноэкранном режиме они не масштабируются под окно и могут быть не видны после выхода из полноэкранного режима.
  3. Нету кнопки на Reddit
  4. В вк при просмотре видео из поста группы вылезает ошибка Uncaught (in promise) VideoDataError: Entered unsupported link
  5. Uncaught (in promise) ReferenceError: url is not defined на сайте X.com и встроенных плеерах

@SashaXser
Copy link
Author

Можешь адаптировать под эту ветку? Там, уже сделан перенос с каллбеков и немного изменена логика работы, но для релиза еще не все сделано и протестировано

Посмотрел код, но не понял, что нужно адаптировать. Кажется, он работает так же, как и в версии 1.5.1. Единственное замечание: можно полностью отказаться от сервера Яндекса, так как он не даст возможности полноценно использовать твой новый backend.

@ilyhalight
Copy link
Owner

Нету кнопки на Reddit

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

Посмотрел код, но не понял, что нужно адаптировать. Кажется, он работает так же, как и в версии 1.5.1.

В целом да, просто нужно сделать, чтобы при мерже не было конфликтов

можно полностью отказаться от сервера Яндекса, так как он не даст возможности полноценно использовать твой новый backend.

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

@ilyhalight ilyhalight closed this Jul 24, 2024
@ilyhalight ilyhalight reopened this Jul 24, 2024
@ilyhalight ilyhalight changed the base branch from master to votjs-integration July 24, 2024 00:55
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