CerebruxCode / shelldio

Ένα απλό shell script για να παίζετε στο Linux, macOS και BSD τερματικό, τους αγαπημένους σας online ραδιοφωνικούς σταθμούς
https://cerebrux.net
GNU General Public License v3.0
33 stars 10 forks source link

Προσθήκη λειτουργίας new_station #106

Closed tasakos-dev closed 3 years ago

tasakos-dev commented 3 years ago

Προσθήκη λειτουργίας new_station η οποία προσθέτει έναν νέο σταθμό στο all_station.txt

cerebrux commented 3 years ago

Από ότι βλέπω ο νέος σταθμός με την επιλογή -n προτίθεται στο all_stations. Αυτό είναι ασύμβατο με την επιλογή -f που έχει το shelldio.

-f, --fresh:    Κατεβάζει εκ νέου την γενική λίστα των ραδιοφωνικών σταθμών με επικαιροποιημένους
            ραδιοφωνικούς σταθμούς, διορθωμένα links αλλά και νέους ραδιοφωνικούς σταθμούς

Π.χ. ο χρήστης προσθέτει έναν νέο σταθμό με την επιλογή -n. Κάποια άλλη στιγμή, πάει και προσθέτει άλλα 10... Μετά από κάνα μήνα, μαθαίνει ότι έχουμε προσθέσει νέους σταθμούς στην εφαρμογή, και λέει "Α ωραία, κάτσε να πάρω τους σταθμούς" και τρέχει shelldio -f... αυτό όπως καταλαβαίνεις πάει και αντικαθιστά το αρχείο all_stations με το νέο, οπότε χάνονται όλες οι προσθήκες που έκανε.

Θα μπορούσε εναλλακτικά η επιλογή -n να δουλεύει στο my_stations το οποίο είναι του χρήστη και το shelldio δεν το πειράζει.

Τι λες ?

zoispag commented 3 years ago

Μετά από κάνα μήνα, μαθαίνει ότι έχουμε προσθέσει νέους σταθμούς στην εφαρμογή, και λέει "Α ωραία, κάτσε να πάρω τους σταθμούς" και τρέχει shelldio -f... αυτό όπως καταλαβαίνεις πάει και αντικαθιστά το αρχείο all_stations με το νέο, οπότε χάνονται όλες οι προσθήκες που έκανε.

Για την ακρίβεια, το update μέσω git δε θα δουλεύει πια γιατί το αρχείο θα έχει conflicts.

tasakos-dev commented 3 years ago

Άρα να κάνω την αλλαγή ώστε να πειράζει το αρχείο my_stationsκαι να κάνω νέο pull request?

zoispag commented 3 years ago

Άρα να κάνω την αλλαγή ώστε να πειράζει το αρχείο my_stationsκαι να κάνω νέο pull request?

Μπορείς να το κανείς και στο ίδιο PR. Δεν χρειάζεται καινούριο! 🙂

cerebrux commented 3 years ago

Ενσωματώθηκε επιτυχώς ! Μπράβο !