ioniodi / site-gr

Moved to ioniodi/sitegr
https://vibrant-haibt-349f89.netlify.com
The Unlicense
3 stars 24 forks source link

Δημιουργία σελίδας για το ωρολόγιο πρόγραμμα #141

Closed AsteriosP closed 4 years ago

AsteriosP commented 4 years ago

Σχετικό Issue

closes #114

το demo με τις αλλαγές μου βρίσκεται εδώ

Σχετικό PR στο minimal-ionio

https://github.com/ioniodi/minimal-ionio/pull/31

Προτεινόμενες Αλλαγές

Υπενθυμίσεις

Σχόλια

Spirosvw commented 4 years ago

Μπράβο @AsteriosP πολύ καλή δουλειά. Θα συμφωνήσω όμως με τον @p15zerv πως ίσως θα ηταν πιο εύχρηστο για την γραμματεία να μπορεί να ανεβάσει ένα αρχείο τύπου pdf ή docx (πιθανό νέο issue?). Παρόλα αυτά, η υλοποίηση αυτή φαίνεται πολύ καλή οπότε θα προχωρήσω και εγώ στο approve όταν μεταφέρεις το timetables.md στο pages του theme.

KonstantinosG commented 4 years ago

Πολύ καλή η υλοποίηση @AsteriosP! Είναι πολύ εύχρηστος ο πίνακας καθώς έχει highlight όταν κάνεις hover με τον κέρσορα. Φαίνεται κατευθείαν δηλαδή η ώρα των μαθημάτων ειδικότερα για αυτά που βρίσκονται προς τις τελευταίες στήλες. Ας γίνει και το τελευταίο βήμα που ανέφεραν και οι υπόλοιποι (@Spirosvw @p15zerv) για να προχωρήσει.

AsteriosP commented 4 years ago

@p15zerv, είναι έτοιμο

provopoulos commented 4 years ago

@p15zerv Κατά λάθος ζήτησα επανεξέταση του αιτήματός σου για αλλαγές ενώ έψαχνα για μια άλλη λειτουργία. Υποθέτω πως μπορείς να το ενημερώσεις ξανά μόλις πραγματοποιηθούν οι αλλαγές που αναφέρθηκαν.

epidrome commented 4 years ago

@AsteriosP βγάζει σύγκρουση σε ένα αρχείο, ίσως θα μπορούσε να αφαιρεθεί και να μπει σε επόμενο χρόνο;

πάντως έχει ενδιαφέρον γιατί βλέπετε στην πράξη πως οι αλλαγές σε πολλά αρχεία σε ένα συνεργατικό έργο είναι η συνταγή για προβλήματα..

@Spirosvw @KonstantinosG @provopoulos @andreanastil σας παρακαλώ για την άμεση αξιολόγηση για να προχωρήσουμε σε ριζική αναμόρφωση του αποθετηρίου

provopoulos commented 4 years ago

@epidrome

Βγάζει σύγκρουση σε ένα αρχείο, ίσως θα μπορούσε να αφαιρεθεί και να μπει σε επόμενο χρόνο;

Νομίζω η επίλυση της συγκεκριμένης σύγκρουσης είναι εύκολη (αν καταλαβαίνω καλά) ωστόσο καλύτερα να το ελέγξει ο @AsteriosP για να είμαστε σίγουροι πριν συνεχίσουμε.

Σε κάθε περίπτωση δεν παρουσιάζεται μήνυμα λάθους 404 στο DEMO με τις τελευταίες αλλαγές όμως τα ορθογραφικά λάθη παραμένουν.