sircharlo / meeting-media-manager

A cross platform app to download and present media for congregation meetings of Jehovah’s Witnesses in any language. Features include automatic media downloading, easy drag-and-drop adding of additional media, and media presentation tools for meetings.
https://sircharlo.github.io/meeting-media-manager/
GNU Affero General Public License v3.0
149 stars 27 forks source link

Error downloading media for meeting #3304

Closed HananiasSQ closed 1 week ago

HananiasSQ commented 1 week ago

What happened?

Updated to the latest and ever since... I get the error downloading media for meeting. So now I can't access the meeting media. As you can see tonight's meeting is the weekday meeting, but MMM says "Watchtower Study", even though, you can clearly see the images are from "Treasures from Gods word" ... something got mixed up in my MMM and I don't know how to fix. Any help would be amazing! I can add videos, but I can't add opening songs on the weekend meeting.

Screenshot 2024-11-14 121805

To Reproduce

  1. Go to '...'
  2. Click on '....'
  3. Do '....'
  4. See error

What did you expect to happen?

I really just want to update the media automatically and have access to meeting media.

Version

24.11.1

What platform are you seeing the problem on?

Windows

Relevant log output

No response

Screenshots

Screenshot 2024-11-14 121805

Additional context

No response

Confirmations

mtdvlpr commented 1 week ago

Hi @HananiasSQ, could you please verify that the website address is correctly configured under Advanced > Danger zone?

mtdvlpr commented 1 week ago

The website address should be "jw.org" (without a language like /en) in almost all cases.

sircharlo commented 1 week ago

I think I'll add some more validation to the baseUrl field. If not a valid two part URL, go to settings and highlight the field.

⛔️ www.site.com ⛔️ site.com/subdomain ⛔️ s.c ⛔️ [empty] ✅ site.com

mtdvlpr commented 1 week ago

I think I'll add some more validation to the baseUrl field. If not a valid two part URL, go to settings and highlight the field.

⛔️ www.site.com ⛔️ site.com/subdomain ⛔️ s.c ⛔️ [empty] ✅ site.com

I wouldn't add too much validation. A proxy server could live on a subdomain for example. This is basically fixed already in the next release: https://github.com/sircharlo/meeting-media-manager/blob/ac14620b30652b9e42792c572218d9d39caa316c/src/stores/current-state.ts#L97

mtdvlpr commented 1 week ago

And this has been improved further with:

https://github.com/sircharlo/meeting-media-manager/blob/ac14620b30652b9e42792c572218d9d39caa316c/src/pages/SettingsPage.vue#L175-L181

sircharlo commented 1 week ago

Fair points.

sircharlo commented 1 week ago

Possibly fixed by https://github.com/sircharlo/meeting-media-manager/pull/3318. Please let me know once v24.11.3 is released, if this fixes the issue for you.

sircharlo commented 1 week ago

Please re-open this issue if the problem persists! @HananiasSQ