Skip to content

Conversation

@freearhey
Copy link
Contributor

Changes:

  • fixed a bug that caused not all issues to be processed during daily update
  • logo width and height are now rounded to integers
  • textarea replaced by input in the logos:remove form

Test results:

npm test           

> test
> jest --runInBand

 PASS  tests/commands/db/validate.test.ts (11.794 s)
 PASS  tests/commands/db/update.test.ts
 PASS  tests/commands/db/export.test.ts

Test Suites: 3 passed, 3 total
Tests:       7 passed, 7 total
Snapshots:   0 total
Time:        15.148 s, estimated 20 s
Ran all test suites.

@freearhey freearhey changed the title Patch 2025.07.2 Patch 2025.07.1 Jul 15, 2025
@Archrootsda
Copy link
Collaborator

Archrootsda commented Jul 15, 2025

@freearhey I don't know if it's a bug. Regarding the feed logos, France24 feeds should display the main logo in apps when the feed doesn't have one. Only the French feed appears with the logo. The others have disappeared. https://iptv-org.github.io/channels/fr/France24
Captura de pantalla de 2025-07-15 23-34-55

@freearhey
Copy link
Contributor Author

@Archrootsda Thanks for pointing that out! I will fix this in the next patch to iptv-org/iptv.

@Alstruit Alstruit requested a review from PopeyeTheSai10r July 16, 2025 03:10
@BellezaEmporium BellezaEmporium merged commit b8cee6c into master Jul 18, 2025
2 checks passed
@BellezaEmporium BellezaEmporium deleted the patch-2025.07.2 branch July 18, 2025 22:03
@freearhey
Copy link
Contributor Author

@Archrootsda I just checked the playlists and it appears that the logos for the feeds are loading correctly. In this case the problem is that the only logo we have for these feeds is in SVG format, which is not supported by most players as I know.

@Archrootsda
Copy link
Collaborator

Thanks for checking. If most of them aren't compatible with the SVG format, I'll try not to add SVG logos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants