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

[KonstantinosG]

Σχετικό Issue

closes #69

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

-- Έχει δημιουργηθεί νέα σελίδα με τα στοιχεία επικοινωνίας με το Τμήμα Πληροφορικής του Ιονίου Πανεπιστημίου. Ο σύνδεσμος της σελίδας "Επικοινωνία" βρίσκεται προσωρινά στο επάνω (βασικό) μενού της ιστοσελίδας.

-- Ο χάρτης περιέχει πολλαπλές πινέζες για τα σημεία ενδιαφέροντος που θα μπορούσαν πιθανόν να βοηθήσουν τους νέους φοιτητές.

..

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

• Έχω ανοίξει από πριν issue για τον καλό συντονισμό του project, το οποίο έχει πάρει το πράσινο φως με την αντίστοιχη ετικέτα
• Έχω δημιουργήσει branch για τις αλλαγές

[epidrome]

@AsteriosP @Spirosvw
αν θέλετε να το δείτε με προτεραιότητα
ώστε να προχωρήσει ο @KonstantinosG στο
επόμενο παραδοτέο του

[epidrome]

some minor comments:

1. index.html is changed with just line feeds?

2. "contact.md" instead of "communication.md"?

3. the content in communication.md looks more like html than liquid-markdown. one idea is that we create some new front-matter variables for the building locations and we move the html-code inside an include file at the theme repo: minimal-ionio? the PR looks OK as it is, but please consider the above code optimisations for your next PR

[NikosAvg]

@KonstantinosG Η σελίδα φαίνεται πολύ ωραία και ο χάρτης δουλεύει πολύ καλά στο δικό μου μηχάνημα.
Θα συμφωνήσω με το σχόλιο του @epidrome για το όνομα του αρχείου αλλά δεν νομίζω ότι είναι και μεγάλο πρόβλημα.
Το ότι το communication.md μοιάζει με html-code και όχι με liquid-markdown εγώ δεν το βλέπω
θα έλεγα ότι έχει ακριβώς την ίδια μορφή με τα υπόλοιπα αρχεία στο φάκελο.
Γενικά πολύ ωραία δουλειά μπράβο.

[AsteriosP]

έχοντας δει το demo (δεν υπάρχει link στο pull request) και τα αρχεία που έχουν αλλαχτεί εγκρίνω τις αλλαγές
ίσως το μόνο το όποιο θα άλλαζα, θα ήταν το index που δεν υπάρχει λόγος να υπάρχουν οι δύο καινούριες σειρές

[epidrome]

@Spirosvw review???

[epidrome]

@KonstantinosG
όπως βλέπεις, στο σημείο 1, στο παραπάνω σχόλιο μου

έστειλες επεξεργασμένο αρχείο, το οποίο δεν χρειαζόταν στο αίτημα σου

δοκίμασα να το διαγράψω με κομιτ εδώ

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

@NikosAvg σε παρακαλώ να προχωρήσεις στον συντονισμό του αιτήματος!

[NikosAvg]

@epidrome Τι εννοείτε όταν λέτε συντονισμό του αιτήματος;

[epidrome]

@NikosAvg ότι χρειάζεται για την σωστή αποδοχή του!
(σχολιασμός, αξιολόγηση, μελέτη)

δεν το λέω-γράφω 1η φορά:

θυμίζω ότι είναι βασικό κριτήριο για την βαθμολόγηση
https://github.com/ioniodi/site-gr/wiki/Grading

Για την άριστη βαθμολόγηση (8-9-10), εκτός από την ποσότητα-δυσκολία των αιτημάτων, θα εκτιμηθεί κυρίως η συνεργατικότητα όπως αυτή αποτυπώνεται από τα τις πρωτοβουλίες σας για νέα θέματα που δεν υπήρχαν, καθώς και από τις πρωτοβουλίες και την ανάλυψη συντονιστικού ρόλου, π.χ., αξιολόγηση-διορθώσεις-αποδοχή σε αιτήματα συναδέλφων.

[NikosAvg]

Ένταξη κατάλαβα

[Spirosvw]

Μετά τις αλλαγές που πρότειναν και οι προηγούμενοι reviewers και την δοκιμή του χάρτη, εγκρίνω τις αλλαγές. Θα συμφωνήσω με τον @NikosAvg για το content του communication.md αλλά το όνομα contact ίσως είναι όντως πιο σωστό. Θα πρότεινα στον @KonstantinosG να περιλαμβάνει και ένα link της σελίδας που θα περιέχει τις αλλαγές για να είναι ευκολότερη η αξιολόγηση της λειτουργικότητας των αλλαγών, σε μελλοντικά pull-requests

[NikosAvg]

@KonstantinosG Για να γίνει δεκτό το pull request προτείνω να αλλάξει το όνομα από communication σε contact και να διορθωθεί το πρόβλημα στο index(Να προσθέσει το αλλαγμένο index όπως πρότεινε ο @epidrome για να διορθωθεί το conflict).

[KonstantinosG]

Λόγω των αυξημένων υποχρεώσεων της προηγούμενης εβδομάδας δεν κατάφερα να προχωρήσω στις διορθώσεις που προτείνετε όλοι στα σχόλιά σας. Ευχαριστώ για το review όλων. Τις επόμενες ημέρες θα προχωρήσω στις αλλαγές και θα κάνω push.

[NikosAvg]

Ωραία @KonstantinosG έγινε δεκτό χωρίς conflict.

[Spirosvw]

Πολύ καλή δουλειά @KonstantinosG ! Πιστεύω ότι είναι έτοιμο για merge.

[KonstantinosG]

Αφήνω και το λινκ για τη διευκόλυνσή σας:

https://confident-brattain-27ead2.netlify.app/index

[AsteriosP]

μετά τις τελευταίες αλλαγές είναι μια χαρά