Project

General

Profile

Segnalazione #72

App polygen

Added by Enrico Zini over 7 years ago. Updated over 7 years ago.

Status:
Chiuso
Priority:
Normale
Assignee:
Start date:
11/23/2016
Due date:
% Done:

0%

Estimated time:

Description

Porting del vecchio plugin polygen

Associated revisions

Revision 4e659e10 (diff)
Added by Mark Caglienzi over 7 years ago

Draft Polygen app's skeleton. refs: #72

Revision 71f6b0b0 (diff)
Added by Mark Caglienzi over 7 years ago

Use reverse_lazy. refs: #72

Revision b39c6fbe (diff)
Added by Mark Caglienzi over 7 years ago

Catch all things in the url, and make some validation in the view. refs: #72

Revision c5fcebfd (diff)
Added by Mark Caglienzi over 7 years ago

Draft to allow to navigate in the path. refs: #72

Revision ff39febf (diff)
Added by Mark Caglienzi over 7 years ago

Show items in the template. refs: #72

Revision 1e6ec61e (diff)
Added by Mark Caglienzi over 7 years ago

Cleanup. refs: #72

Revision ff712d6c (diff)
Added by Mark Caglienzi over 7 years ago

Get separated variables. refs: #72

Revision 06cb86f0 (diff)
Added by Mark Caglienzi over 7 years ago

Implement the view. refs: #72

Revision d79be1e6 (diff)
Added by Mark Caglienzi over 7 years ago

Template for polygen:list view. refs: #72

Revision 23dec8fc (diff)
Added by Mark Caglienzi over 7 years ago

Add a couple of auxiliary templatetags to join strings and paths in the templates. refs: #72

Revision c09d1fc4 (diff)
Added by Mark Caglienzi over 7 years ago

To avoid ugly urls with two concatenated '/' characters, get all path in one variable. refs: #72

Revision 3918e7c7 (diff)
Added by Mark Caglienzi over 7 years ago

Use utility_tags and concatenate the paths with os_path_join. refs: #72

Revision c284396f (diff)
Added by Mark Caglienzi over 7 years ago

Use only buttons to go back. refs: #72

Revision 35380a25 (diff)
Added by Mark Caglienzi over 7 years ago

Add split_path() method to the view. refs: #72

Revision 94f18015 (diff)
Added by Mark Caglienzi over 7 years ago

Use the method. refs: #72

Revision 3bcbc416 (diff)
Added by Mark Caglienzi over 7 years ago

Fix almost everything in 'polygen:list' view. refs: #72:

  • Move path splitting in the dispatch() method * Redirect to polygen root with error message in case of path not
    found in tree (just a 404 was ugly from the UI/UX point of view,
    even if it's better from the HTTP point of view. Just in case,
    the call is in the code, commented out) * Ugly 'hack' that creates self.root_tree also in List.dispatch()
    (see the FIXME comment in the code/diff) and allows the behaviour
    explained at the previous point

Revision c603974a (diff)
Added by Mark Caglienzi over 7 years ago

Cleanup. refs: #72

Revision 7234407f
Added by Mark Caglienzi over 7 years ago

Merge branch 'master' into t72. refs: #72

Revision fec2f1c7 (diff)
Added by Mark Caglienzi over 7 years ago

Sort these lists. refs: #72

Revision 5e796187 (diff)
Added by Christopher R. Gabriel over 7 years ago

refs #72

small ui improvements

Revision 7ec0721a (diff)
Added by Enrico Zini over 7 years ago

Refactoed without code duplication in dispatch. Refs #72

History

#1

Updated by Mark Caglienzi over 7 years ago

  • Status changed from Nuovo to In elaborazione
  • Assignee set to Mark Caglienzi
#2

Updated by Mark Caglienzi over 7 years ago

  • Status changed from In elaborazione to Commenti
  • Assignee changed from Mark Caglienzi to Christopher R. Gabriel
5e7b6e45:
  • Voce visibile nella sidebar, con la sua icona
  • La home dell'app è l'elenco delle lingue, a cui segue, dopo clic, l'elenco delle grammatiche di quella lingua
  • Nella pagina dell'elenco delle grammatiche c'è un bottone back per tornare indietro
  • Cliccando su una grammatica si genera una entry per quella grammatica, e nella pagina si aggiunge anche il bottone per generarne un'altra
  • Se si scrive un'url inesistente, messaggio di errore e redirect alla home dell'app (restituire semplicemente 404 è più corretto dal punto di vista dello standard HTTP e REST, ma dal punto di vista della UI la pagina è davvero brutta)
  • Magari si può fixare dopo aver rifatto la pagina 404 standard. Altro inconveniente è che il messaggio di errore in questo caso viene mostrato dopo che si ricarica una pagina "buona" e non nella 404 stessa. Questo potrebbe confondere l'utente.

Mergiato in master (dopo merge di master in t72 e risoluzione di un conflitto facile in octonet/urls.py) e push.

#3

Updated by Christopher R. Gabriel over 7 years ago

  • Assignee changed from Christopher R. Gabriel to Enrico Zini

Per me ok, ho aggiungo qualche fix per la UI e basta.

Riassegno ad Enrico per la review della implementazione, per parlarne poi con Mark.

#4

Updated by Enrico Zini over 7 years ago

  • Status changed from Commenti to Chiuso

Direi che ci sia tutto. Ho pushato in master un refactoring della dispatch per evitare duplicazione di codice con OctonetMixin

Also available in: Atom PDF