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

Update channels.csv #12583

Merged
merged 40 commits into from
Aug 5, 2024
Merged

Update channels.csv #12583

merged 40 commits into from
Aug 5, 2024

Conversation

ProPontifex
Copy link
Contributor

See title

@ProPontifex
Copy link
Contributor Author

Please try and merge: Belleza, when you have the time, could you please correct? Thank you.

Small revert on changes.
Fixed ERT1 naming.
Fixed 002RadioTV.do incorrect info.
Copy link
Contributor

@Alstruit Alstruit left a comment

Choose a reason for hiding this comment

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

Are you attempting to switch the countries from where the broadcast originates to where it is received?

@ProPontifex
Copy link
Contributor Author

If I change the country of origin for a channel, it always defaults to where it originates. If I made a mistake, feel free to correct :)

@ProPontifex
Copy link
Contributor Author

(Belleza?) Would some please be kind enough to correct and merge these changes. Thank you in advance.

@BellezaEmporium
Copy link
Contributor

I have been summoned it seems. I'm checking the merge conflicts tomorrow.

@ProPontifex
Copy link
Contributor Author

Thank you, sir

@BellezaEmporium
Copy link
Contributor

Resolved.

9248 "GalvestonCountyTV.us" has the wrong broadcast_area "c/US-TX"
9406 "GFNTV.us" has an invalid categories "entrertainment"
9464 "Glendale11AZ.us" has the wrong broadcast_area "c/US-AZ"
8830 Food52.us: "owners[0]" with value "Food52, Inc" fails to match the required pattern: /^[^",]+$/
9248 GalvestonCountyTV.us: "broadcast_area[0]" with value "c/US-TX" fails to match the required pattern: /^(s/[A-Z]{2}-[A-Z0-9]{1,3}|c/[A-Z]{2}|r/[A-Z0-9]{2,7})$/
9248 GalvestonCountyTV.us: "broadcast_area" does not contain 1 required value(s)
9464 Glendale11AZ.us: "broadcast_area[0]" with value "c/US-AZ" fails to match the required pattern: /^(s/[A-Z]{2}-[A-Z0-9]{1,3}|c/[A-Z]{2}|r/[A-Z0-9]{2,7})$/
9464 Glendale11AZ.us: "broadcast_area" does not contain 1 required value(s)

@ProPontifex
Copy link
Contributor Author

Corrections made: thank you

@BellezaEmporium
Copy link
Contributor

BellezaEmporium commented Aug 5, 2024

9741 GreenChannel.jp: "owners[0]" with value "Satellite Broadcasting Foundation for Horseracing, Agriculture, Forestry and Fisheries" fails to match the required pattern: /^[^",]+$/
9742 GreenChannel2.jp: "owners[0]" with value "Satellite Broadcasting Foundation for Horseracing, Agriculture, Forestry and Fisheries" fails to match the required pattern: /^[^",]+$/

These should be wrapped in "".

EDIT : They are wrapped, though it seems the bot still didn't like it.

@ProPontifex
Copy link
Contributor Author

Thanks Belleza: I just took the commas out shrug

Copy link
Contributor

@BellezaEmporium BellezaEmporium left a comment

Choose a reason for hiding this comment

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

OK, all good now. No touching until PR gets merged.

@ProPontifex
Copy link
Contributor Author

Copy that: thank you

@Dum4G Dum4G merged commit e2fc83d into iptv-org:master Aug 5, 2024
1 check passed
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.

4 participants