Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vorschlag #1

Open
kfessel opened this issue Nov 24, 2022 · 4 comments
Open

Vorschlag #1

kfessel opened this issue Nov 24, 2022 · 4 comments

Comments

@kfessel
Copy link

kfessel commented Nov 24, 2022

            if (Edrys.module.config) {
                let md = "" + Edrys.module.config.?md || ""
                md += edrys.module[edrys.role.toLowerCase()+'Config']?.md || ""
                render(md)
            }

an Stelle von

if (Edrys.module.config) {
switch (Edrys.role) {
case "teacher": {
render(Edrys.module.config.teacher || Edrys.module.config.md)
break
}
case "student": {
render(Edrys.module.config.student || Edrys.module.config.md)
break
}
case "station": {
render(Edrys.module.config.station || Edrys.module.config.md)
break
}
}
}

@andre-dietrich
Copy link
Contributor

Wie ist das? Man kann mit Yaml auch nur Strings definieren, dann kann man das .md auch umgehen...

let content = Edrys.module.config || ""
content += "\n\n" + Edrys.module[Edrys.role.toLowerCase()+'Config'] || ""
render(content)

@kfessel
Copy link
Author

kfessel commented Nov 24, 2022

ich würde das einpacken in "md" oder so, dadurch bleibt die Möglickeit bestehen zusätzliche Konfiguration unterzubringen

@kfessel
Copy link
Author

kfessel commented Nov 24, 2022

ich hatte sogar überlegt ob man irgendwas machen kann wie

"""
let c = ""

for i in 1..3 {
c += Edrys.module.config?.md[i] || ""
c += Edrys.module[Edrys.role.toLowerCase()+'Config']?.md[i] || ""
}
"""

so könnte man mehrere Abschnitte kombinieren

@kfessel
Copy link
Author

kfessel commented Nov 24, 2022

ich habe das gerade getestet vielleicht ist das so doch ganz gut da man das markdown auch ohne einrückung in der config haben kann

leider hat bei test der ist beim test der leeren config ein "undefined" erschienen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants