Skip to content

a (really) small update to polish playlist - database (part 2/3) - #20574

Merged
Dum4G merged 8 commits into
iptv-org:masterfrom
Jxpl9:master
Sep 1, 2025
Merged

Dum4G merged 8 commits into
iptv-org:masterfrom
Jxpl9:master

Conversation

@Jxpl9

@Jxpl9 Jxpl9 commented Aug 31, 2025 •

Copy link
Copy Markdown
Collaborator

This is a Part 2 of the project.
Thanks for @Dum4G for providing the missing IDs.
State of this project:
✅ Copy all Rakuten.tv channel names one by one.
✅ Copy all Rakuten.tv channel logos one by one. I have all the links for the logos on my computer.
✅ Copy all Rakuten.tv channel m3u8s one by one.
✅ Separate the pl.m3u playlist into three files.
✅ Add missing channels to the playlists.
✅/❓ Add missing IDs.
❌ Add logos to the database.
🟨 Add missing 'Poland' feeds to the database.

@freearhey freearhey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure that the number of columns in the file has not changed and that all rows end with CRLF.

@Jxpl9

Jxpl9 commented Aug 31, 2025

Copy link
Copy Markdown
Collaborator Author

wait @freearhey , so can I edit the file with libreoffice calc, or I must edit it with program like notepad++?

@freearhey

Copy link
Copy Markdown
Member

any text editor will work for editing files, but if you're using Linux or macOS, you'll need to change the line endings to the Windows version. some programs have an option for this. otherwise, you can use a utility such as unix2dos.

@Jxpl9

Jxpl9 commented Aug 31, 2025

Copy link
Copy Markdown
Collaborator Author

any text editor will work for editing files, but if you're using Linux or macOS, you'll need to change the line endings to the Windows version. some programs have an option for this. otherwise, you can use a utility such as unix2dos.

I'm using on this pc w11, so i think I don't have this problem, but even earlier when I was using gentoo as far as I remember I didn't have that problem

@Jxpl9
Jxpl9 requested a review from freearhey August 31, 2025 15:59
@freearhey

Copy link
Copy Markdown
Member

When attempting to correct the line endings, a new problem appeared:

> db:validate
> tsx scripts/commands/db/validate.ts


feeds.csv
 72    "13thStreet.au@Plus2" id "Plus2" must be derived from the name "2"
 158   "2MMonde.ma@Plus1" id "Plus1" must be derived from the name "1"
...

Your editor apparently considered feed names such as "+1” and “+2” to be numbers and simply removed the “+” symbol from them.

@BellezaEmporium

Copy link
Copy Markdown
Contributor

CRLF fixed. Will add something to the GitHub Actions to verify that all files are CRLF.

@freearhey freearhey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The plus symbol is still missing.

@Archrootsda

Copy link
Copy Markdown
Contributor

any text editor will work for editing files, but if you're using Linux or macOS, you'll need to change the line endings to the Windows version. some programs have an option for this. otherwise, you can use a utility such as unix2dos.

@freearhey according to what you explain, (under the GNU/LINUX and Mac OS operating systems) once downloaded and edited, we have to convert the csv file with the command "unix2dos archive.csv" and upload it to the repository (fork), right?
https://lindevs.com/install-dos2unix-on-ubuntu

@freearhey

Copy link
Copy Markdown
Member

If your editor allows you to switch line endings in files, you don't need to do anything extra. In the case of LibreOffice Calc, as I understand it, there is just no such option.

@Jxpl9
Jxpl9 requested a review from freearhey September 1, 2025 13:37
@Jxpl9 Jxpl9 changed the title a (really) small update to polish playlist - database a (really) small update to polish playlist - database (part 2/3) Sep 1, 2025
@Jxpl9

Jxpl9 commented Sep 1, 2025

Copy link
Copy Markdown
Collaborator Author

okey, I think it's time to wait for part 3

@Dum4G
Dum4G enabled auto-merge September 1, 2025 22:47
@Dum4G
Dum4G merged commit b0afa8f into iptv-org:master Sep 1, 2025
2 checks passed
@Archrootsda

Copy link
Copy Markdown
Contributor

@Jxpl9 The added channels don't have their logos. The logos.csv file hasn't been modified.

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