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

Δεν γίνεται έλεγχος εγκυρότητας του αρχείου #16

Closed
drpaneas opened this issue Sep 11, 2020 · 3 comments · Fixed by #32
Closed

Δεν γίνεται έλεγχος εγκυρότητας του αρχείου #16

drpaneas opened this issue Sep 11, 2020 · 3 comments · Fixed by #32
Labels
bug Something isn't working enhancement New feature or request

Comments

@drpaneas
Copy link

Αν παω και γράψω ασυναρτησίες ή ακόμα αν κάνω ένα λάθος στην συμπλήρωση του αρχείου.txt το shelldio δεν το παίρνει χαμπάρι. Θα ήθελα να μπορεί να κάνει check το validity του αρχείου πριν το χρησιμοποιήσει.

@cerebrux cerebrux added bug Something isn't working enhancement New feature or request labels Sep 11, 2020
@JohnGavr
Copy link

Τα link για τα web radio δεν έχουν συγκεκριμένη δομή ώστε να προστεθεί κάποιος έλεγχος. Λόγω του ότι η κάθε γραμμή στα txt files είναι
Όνομα σταθμού,link σταθμού
Θα μπορούσαμε να ελέγχουμε αν μετά το κόμμα είναι url.

@drpaneas
Copy link
Author

drpaneas commented Sep 14, 2020

Τα link για τα web radio δεν έχουν συγκεκριμένη δομή ώστε να προστεθεί κάποιος έλεγχος.

Ναι αυτο λεω. Θα πρεπει να εχουν δομή. πχ αυτο το αρχειο ειναι εγγυρο?

sdf,link
sds,   link
asd,,,link

Αν οχι, θα ηθελα να μου βγαλει σχετικο σφαλμα το script: "Το αρχείο με τους σταθμούς έχει προβλημα". Για να μην καθεσαι να γραφεις parser, χρησιμοποιήσε μία έτοιμη data structure, πχ toml.

Θα μπορούσαμε να ελέγχουμε αν μετά το κόμμα είναι url.

Ναι, γιατι οχι. Σε 2ο βήμα ακούγεται καλό check.

@cerebrux
Copy link

cerebrux commented Sep 17, 2020

Στην νέα έκδοση δεν θα χρειάζεται ο χρήστης να επεμβαίνει με το χέρι τους για τους σταθμούς. Το σωστό είναι να μας λέει να βάλουμε τον σταθμός και έπειτα αυτός θα τρέχει:

shelldio --fresh

και θα του έρθουν νέοι σταθμοί. Μετά μπορεί να τους προσθέσει στα αγαπημένα του με το shelldio --add

fixed with ef95fe9

@cerebrux cerebrux reopened this Sep 19, 2020
@cerebrux cerebrux linked a pull request Sep 19, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants