stgm / course-site

Course website that serves Markdown from a git repository. A great tool for teachers who happen to think like programmers.
5 stars 8 forks source link

Relatieve links van geïmporteerde repos worden niet correct verwerkt #401

Open simonpauw opened 3 years ago

simonpauw commented 3 years ago

Bijvoorbeeld:

Vanuit materials.yml impoerteer ik sp1-python als python

In de python directory zit de pagina en/dictionaries/index.md. Het pad hiernaar wordt /python/en/dictionaries/, zoals verwacht.

Maar, in de de index.md zit een relatieve link: basic-operations/ Dit wordt omgezet naar /materials/python/en/dictionaries/basic-operations/ in plaats van /python/en/dictionaries/basic-operations/

Dit maakt het nu onmogelijk om relatieve links te gebruiken binnen de opdrachten repo's

stgm commented 3 years ago

Goeie! Is op te lossen.

stgm commented 3 years ago

Correctie: is niet zo goed op te lossen. Relatieve links zijn sowieso finicky in HTML omdat er een verschil is als de huidige URL met een slash eindigt of niet.

Dus het ligt voor de hand om sowieso absolute URLs te gebruiken. De vraag is dan of je die nog automatisch wil prefixen.

Als je het dus zo doet dan kun je niet meer teruglinken van een subrepo naar andere delen van de cursus. Denk dat dat wel tot problemen kan leiden?

Dus ik zou zeggen dat er twee opties zijn:

  1. Alles helemaal handmatig linken.
  2. Relatieve links prefixen, dus jij linkt naar en/dictionaries/basic-operations zonder slash en dan komt er automatisch /python/ voor. Absolute links worden met rust gelaten.

Dat laatste is makkelijk aan te passen maar is het goed werkbaar?