Skip to content
This repository has been archived by the owner on Jun 25, 2020. It is now read-only.

Ανανέωση σελίδας "Επικοινωνία" #144

Merged
merged 8 commits into from
Jun 1, 2020

Conversation

KonstantinosG
Copy link
Collaborator

@KonstantinosG KonstantinosG commented May 29, 2020

Σχετικό Issue

closes #142

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

  • Μετακίνηση της σελίδας επικοινωνίας από το πάνω μέρος της οθόνης στο sitemap και στην κατηγορία "Χρήσιμοι Σύνδεσμοι"

  • Εισαγωγή συνδέσμων που μεταφέρουν το χρήστη στο χάρτες της Google με μία πινέζα στην τοποθεσία που επέλεξαν. Οι σύνδεσμοι είναι εντός της σελίδας "Επικοινωνία" κάτω ακριβώς από τον χάρτη.

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

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

Σημειώσεις:

  • Απαιτείται η ολοκλήρωση του PR από το minimal-ionio όπου βρίσκεται το αρχείο που ανανεώθηκε με τουσ συνδέσμους περιήγησης (communication-info.html).
  • Demo σύνδεσμος

Copy link
Collaborator

@p15zerv p15zerv left a comment

Choose a reason for hiding this comment

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

Μηδαμινές οι αλλαγές εδώ, οπότε μια χαρά. Το PR εξαρτάται ουσιαστικά από την ποιότητα των αλλαγών στο theme, όπου κι εκεί φαίνονται οκ.

Όπως σχολίασα και στο theme, γίνεται πλέον επιτακτική η ανάγκη για τη δομή #132, μιας και πλέον επαναλαμβάνονται οι συντεταγμένες.

Βέβαια, ας είμαστε λίγο υπομονετικοί για το merge, διότι ακόμη δεν έχει πάρει καν greenlight το issue.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Αφησα review στο PR του θεματος. Οπως ειπε ο @p15zerv δεν υπαρχουν αλλαγες εδω.

Για το θεμα του greenlight θα πω οτι ειμαστε στα τελευταια 1-2 PR του αποθετηριου μπορουμε πιστευω να προχωρησει αφου τα ειπαμε και στη συναντηση κιολας.

Copy link
Collaborator

@AsteriosP AsteriosP left a comment

Choose a reason for hiding this comment

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

μικρές οι αλλαγές όποτε και από εμένα είναι Approve

Copy link
Collaborator

@korinaal korinaal left a comment

Choose a reason for hiding this comment

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

Πολύ μικρή η αλλαγή. Δε βρίσκω το λόγο να μην γίνει merge.

@Spirosvw Spirosvw merged commit ab3f50e into ioniodi:master Jun 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants