Skip to content
Snippets Groups Projects

WIP: Reforma importů

Closed Martin Mareš requested to merge mj/import into master

Ze systému importů rostl pěkný bastl: spousta věcí byla implementovaná velmi podobně na více místech a bylo pracné přidávat nové typy importů. Také původní předpoklad, že budeme umět jen jeden formát souboru, se ukázal být příliš naivním.

Pokusil jsem se importy zobecnit, značně to prokouklo. Teď existuje jedna jediná implementace UI k importům, jedna společná dokumentace pro všechny druhy importů a jeden generátor šablon. Formát souboru se vybírá explicitně.

Postupně doplním další drobnosti jako ignorování prázdných řádků.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 105 105
    106 106 tab.append('</table>')
    107 107 if self.show_downlink:
    108 tab.append("<p>Stáhnout jako <a href='?format=csv'>CSV</a> nebo <a href='?format=tsv'>TSV</a>.")
    108 tab.append("<p>Stáhnout jako <a href='?format=cs_csv'>CSV</a> nebo <a href='?format=tsv'>TSV</a>.")
  • Prošel jsem a mimo okomentovaných drobností/návrhů jsem nic podivného nenašel. A líbí se mi, že se neopakuje dokola skoro ten stejný kód :)

  • Jiří Setnička approved this merge request

    approved this merge request

  • Díky za review!

  • Martin Mareš added 1 commit

    added 1 commit

    • c2dc8516 - Import: Hezčí jména souborů s šablonami

    Compare with previous version

  • Martin Mareš added 1 commit

    added 1 commit

    • 147a2f7d - Tabulky: Obecný download link umí všechny 3 formáty

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading