Možnost stáhnout si řešení pojmenovaná podle řešitele #26

Merged
zelvuska merged 3 commits from jmena-do-priloh-reseni into master 2023-02-27 20:48:27 +01:00
Owner

Přidává checkboxík a kus JS, který pojmenuje při stažení souboru z odevzdávátka přidá před název souboru i jméno prvního řešitele a ID řešení. To umožní snadno stáhnout více řešení, aniž by se soubory jmenovaly stejně a aby šly přiřadit zpět k řešitelům i pokud ve svých souborech neuvedli jméno.

Nefunguje mi to na lokálním webu, moje pozorování říká, že Django přidává hlavičku Content-Disposition, která minimálně ve Firefoxu přebíjí <a download=…>, obsahuje-li filename.

Ref: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a#download

Používám na to data- atributy, dokumentace třeba na https://developer.mozilla.org/en-US/docs/Learn/HTML/Howto/Use_data_attributes

Zatím je to trochu PoC, takže je to trošičku vizuálně hnusný. A taky jsem to neotestoval end-to-end. (Vím, že se atribut download přiřazuje správně, vím, že když si v devtools na produkci download nastavím, tak že to pak udělá zhruba správnou věc, ale dohromady jsem to nezkoušel :-))

Přidává checkboxík a kus JS, který pojmenuje při stažení souboru z odevzdávátka přidá před název souboru i jméno prvního řešitele a ID řešení. To umožní snadno stáhnout více řešení, aniž by se soubory jmenovaly stejně a aby šly přiřadit zpět k řešitelům i pokud ve svých souborech neuvedli jméno. Nefunguje mi to na lokálním webu, moje pozorování říká, že Django přidává hlavičku `Content-Disposition`, která minimálně ve Firefoxu přebíjí `<a download=…>`, obsahuje-li filename. Ref: <https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a#download> Používám na to `data-` atributy, dokumentace třeba na <https://developer.mozilla.org/en-US/docs/Learn/HTML/Howto/Use_data_attributes> Zatím je to trochu PoC, takže je to trošičku vizuálně hnusný. A taky jsem to neotestoval end-to-end. (Vím, že se atribut `download` přiřazuje správně, vím, že když si v devtools na produkci `download` nastavím, tak že to pak udělá zhruba správnou věc, ale dohromady jsem to nezkoušel :-))
ledoian added 1 commit 2023-02-22 07:30:14 +01:00
Budu zlobit a vysvětlení napíšu až do pull-requestu. Tady to IMHO čtu
jen já :-P

Pull-Request-URL: #26

(Adresa platí, pokud mě nikdo nepředběhne :-P)
zelvuska reviewed 2023-02-22 10:10:02 +01:00
@ -10,1 +11,4 @@
<script src="{% static 'odevzdavatko/check_for_detail.js' %}"></script>
<script type="text/javascript">
$(document).ready(function () {
console.log("I am ready!");
Owner

Tohle tu chceme nechávat?

Tohle tu chceme nechávat?
Author
Owner

Jeejda… I mean, stejně to nikdo nečte, ale já ta logování smažu :-)

Jeejda… I mean, stejně to nikdo nečte, ale já ta logování smažu :-)
Owner

Ok.

Ok.
ledoian added 1 commit 2023-02-27 20:30:06 +01:00
zelvuska added 1 commit 2023-02-27 20:44:43 +01:00
zelvuska merged commit bec500d61a into master 2023-02-27 20:48:27 +01:00
zelvuska deleted branch jmena-do-priloh-reseni 2023-02-27 20:48:27 +01:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: mam/mamweb#26
No description provided.