yumata / lampa-source

GNU General Public License v2.0
181 stars 58 forks source link

Есть небольшой вопрос + Баг с сохранением последней страницы #132

Open Boria138 opened 1 year ago

Boria138 commented 1 year ago

Задавать вопросы через ошибки на github не очень хорошо но вы не оставили своих контактов так что придётся так вопрос: Почему вы удалили мой код PWA ? Конечно там были ошибки как например ошибки с путями в manifest.json честно ума не приложу как я такое проморгал Возможно вы удалили его потому что для работы PWA на мобильныз устройствах нужен https ? из-за чего все плагины ломаются т.к. загружены на http сайтах а загружать http контент на https зайтах нельзя но это легко фиксится добавлением мета тега который заставляет все http запросы переадрисовываться на https при тааком раскладе ломается только этот список плагинов (MODS's,Lampac 4k, Онлайн - Prestige, Телевидение by Skaz, Онлайн просмотр @kmvrus, Переключение между парсерами, Стиль кнопок, Tracks.js). Все остальные исправно работают

Ну и что бы это хоть как-то было похоже на ошибку добавлю что в ходе тестов своего форка на своём хосте я обнаружил что если главной страницей стоит Последняя то сайт показывает страницу с ошибкой изначально я думал что это моя проблема но на lampa.mx ошибку удалось воспроизвести переключив главный источник на Cub если стоит TMDB то этого бага нет в то время как на моём хосте этот баг исправляется переключением с TMDB на Cub Screenshot_2023-02-09-18-31-44-93_ffb2f5e1b976ff98cfc94f359fbce8de

yumata commented 1 year ago

Там было проблема с sw.js я во первых не понял зачем оно нужно? И во вторых, вызвало ошибки и что-то непонятное творилось с кешем, так как я не понял что оно делает, я удалил этот код.

С минифестом и иконками тоже не понял, те же иконки что в лампе, но по краям добавлен отступ, зачем оно? Я проверить не мог как оно бы смотрелось, поэтому тоже удалил))

yumata commented 1 year ago

Да и в целом не понял, зачем этот PWA? С браузера же можно добавить иконку на рабочий стол и лампа будет работать как приложение, зачем все эти манипуляции?

Boria138 commented 1 year ago

Не знаю есть ли смысл отвечать потому что как я уже по моему где-то писал как оказалась для работы pwa нужно https соединение которое ломает некоторые плагины опять таки список я по моему где-то писал так что я просто захостил свой форк с поддержкой PWA но раз уж вы спросили ответчу

1) sw.js он нужен для написания стратегий кеширования более подробно можно почитать вот здесь это вообше очень универсальная штука которая умеет много чего но я написал там просто простенькую стратегию кеширования которая кеширует список файлов из массива и всё а ошибки в консоли не влияют на работоспособность сайта я уже более двух недель использую установленное через chrome pwa и кроме того что не работает filmix проблем не было

2) Про иконки возможно их не нужно было делать так много для генерации иконок я пользовался сайтом pwa builder который рекомендует гугл

3) Ну и наверное самое главное в чём смысл этого всего всё началось с того что я захотел посмотреть фильмы на ноутбуке на linux через браузер это было не особо удобно да и горазда проще просто наживать на икону на рабочем столе чем заходить на сайт изначально я написал приложение на electron js работало в принципе по тому же принципу что и официальные приложение на винде но вечно вылазили какие-то баги потом вспомнил про существование PWA написал рабочий (как мне тогда казалось) форк решил сделать pull request ведь PWA работает почти во всех браузерах кроме пк Firefox даже safari который много чего не поддерживает умеет в PWA ну а различия между созданием ярлыка и установкой PWA по сути разве что в том что ярлык запускает браузер с сайтом а PWA это отдельное приложение у которого горазда больше возможностей хотя из всех этих возможностей я использую разве что управление отображением там например в manifest параметр display

В общем если захотите можете разобраться в теме более подробно например при помощи этой или этой статьи и уже сами решите надо вам это или нет а на этом обсуждение полагаю можно закончить а то я и так пишу вопросы в баги (написать какую нибудь почту для связи на такой случай было бы неплохо телеграм канал как по мне немного по другой тематике )

Nemiroff commented 1 year ago

Главная проблема: Как завести TorrServe в локалке по https

Boria138 commented 1 year ago

Главная проблема: Как завести TorrServe в локалке по https

А об этом я даже не думал из-за того что не пользоваюсь торрентами ну я может быть поломаю голову как это всё довести до ума и сделать удобным для всех и сделаю новый pull request уже продуманный или оставлю это просто у себя на github больше для личного пользования