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

Merged
zelvuska merged 3 commits from jmena-do-priloh-reseni into master 1 year ago
ledoian commented 1 year ago
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 1 year ago
32dc97e1c4 Možnost přidat si do názvů příloh řešení i jména řešitele
zelvuska reviewed 1 year ago
<script src="{% static 'odevzdavatko/check_for_detail.js' %}"></script>
<script type="text/javascript">
$(document).ready(function () {
console.log("I am ready!");
Poster
Owner

Tohle tu chceme nechávat?

Tohle tu chceme nechávat?
ledoian commented 1 year ago
Poster
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 1 year ago
zelvuska added 1 commit 1 year ago
zelvuska merged commit bec500d61a into master 1 year ago
zelvuska deleted branch jmena-do-priloh-reseni 1 year ago
The pull request has been merged as bec500d61a.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.