Skip to content

Προσθήκη φίλτρων lua και προσθήκη των προπτυχιακών μαθημάτων στο αρχείο ch03.txt του φακέλου text#9

Merged
epidrome merged 2 commits intoioniodi:masterfrom
nkanagno:master
Mar 16, 2024

Conversation

@nkanagno
Copy link
Contributor

Σχετικό Issue

closes #4

Σχετικό Pull Request

Σύνδεσμος σε σχετικό Pull Request στο all_collections που έχετε στείλει και πρέπει να επεξεργαστούμε σε συνδυασμό με αυτό εδώ

DEMO LINK

Σύνδεσμος σε προσωπικό issue που περιέχει το pdf βιβλίο demo

Συνεργάτες

@Axileaszervos @inf2021090 @epidrome
Οι συνεργάτες θα κάνουν σχόλια ή θα αξιολογήσουν τις αλλαγές, κάντε mention @ τουλάχιστον δύο, όχι μέλη της ομάδας σας, αλλά κάποιους που δουλεύουν σε σχετικά θέματα, ώστε να πάρετε αντικειμένικη ανατροφοδότηση.

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

Προσθήκη φίλτρων .lua αρχείων που να παιρνουν ως παράμετρο το μάθημα σε μορφή .md που θα παίρνει από το all_collections και να εμφανίζει στο βιβλίο τον τίτλο (σε μορφή header 4 ####) και το περιεχόμενό του, χωρισμένα στο txt ανά εξάμηνο στο υποκεφάλαιο Περιεχόμενο Μαθημάτων της Ενότητας ΠΡΟΠΤΥΧΙΑΚΕΣ ΣΠΟΥΔΕΣ. Τα 4 διαφορετικά φίλτρα θα χωρίζονται σε 4 κατηγορίες:
- Κορμού
- Επιλογής
- Κατεύθυνσης Πληροφοριακών Συστημάτων
- Κατεύθυνσης Πληροφορική - Ανθρωπιστικές και Κοινωνικές Επιστήμες
Links για τα παρακάτω αρχεία:

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

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

@epidrome
Copy link
Member

@nkanagno

  1. ο κώδικας φαίνεται πως έχει πολύ μεγάλη επικάλυψη στα τέσσερα αρχεία

  2. Ιδανικά, θα θέλαμε ένα αρχείο κώδικα μόνο που εξετάζει το πεδίο του μαθήματος

  3. Πρακτικά η λύση που δίνεις δημιουργεί επανάληψη της ίδια πληροφορίας (είδος μαθήματος) που πλέον είναι διαθέσιμη σε δύο σημεία, στο αρχείο του μαθήματος και ως κλάση στο αρχείο με την λίστα των μαθημάτων, γεγονός που σίγουρα θα δημιουργήσει ασυνέπεια στο μέλλον όταν ένα μάθημα αλλάξει είδος.

@epidrome epidrome merged commit feb0256 into ioniodi:master Mar 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants