Skip to content
This repository has been archived by the owner on Dec 9, 2019. It is now read-only.

Talk e Subcommunity #19

Merged
merged 10 commits into from
Feb 10, 2015
Merged

Talk e Subcommunity #19

merged 10 commits into from
Feb 10, 2015

Conversation

leriomaggio
Copy link
Contributor

Main Change:

  • Spostato il campo sub_community da p3.P3Talk a conference.Talk (segue una PR anche per conference) - con relativo script di migrate south.
  • In più, allineate tutte le info di submission con il CfP (tipi di contributi, time slots).
  • Ancora, corretti alcuni bug nei template "profile" e "talk" (con qualche leggera variazione al css)
  • Infine, modificate alcune impostazioni in settings e requirements per effettuare correttamente il processo di setup.

Tutti i dettagli, sono nei commenti dei commit.

Nella PR c'è anche una vecchia modifica alla pagina degli hotel per un fix delle API GMaps sui controlli di zoom della mappa.

leriomaggio and others added 6 commits April 11, 2014 17:42
I had to move 'conference' before 'p3' to allow migration (due to depencency injected by P3Talk). However, this would break the template resolution mechanism
(exp. related to the conference app). To this end, a new folder has been added in the TEMPLATE_FOLDERS directive.
…field (now included in the Talk model of the `conference` app).

This commit reflects all the changes applied to those classes.

Additional changes to forms:
- Talk time slots now are aligned with those declared in the CfP
- Talk Tracks and Talk Types have been aligned as well.

Moreover, the migration script is also provided (generated by south).
Thus, to correctly apply the modification to the db, please type:

`python manage.py migrate p3`
…on to be displayed completely messy (due to some tags not correctly closed)

Modified and Customised some CSS directives to improve the UI and layout.
@dvd0101
Copy link
Contributor

dvd0101 commented Jan 26, 2015

Perché spostare il campo sub_community in conference? è un dato specifico di pycon per questo stava in p3.

Le altre modifiche invece vanno bene

@leriomaggio
Copy link
Contributor Author

Per quello che ho intuito, conference è responsabile di tutto il processo di gestione di talk (schedule e submissions), mentre p3 gestisce gli aspetti della conferenza.
Seguendo questo razionale, ho inteso il campo sub_community più come "estensione" di (conference.)Talk che come info specifica di PyCon Sei (e, quindi, p3).
In altre parole, la classe P3Talk l'ho intesa come proxy verso conference. Tutte le info di un talk sono (e restano, in questo modo) in conference.talk.
In questo modo si ottiene:

  • una gestione più lineare del processo di salvataggio dei talk (risolvendo alcuni problemi di salvataggio e duplicazione delle sottomissioni);
  • la info è compilant con i vecchi talk, in quanto il campo è empty di default;
  • è possibile abilitare la visualizzazione della info nel Back-end di admin con pochissimo effort, senza modificare admin-forms e admin-manager

Spero di essermi spiegato ;)

@dvd0101
Copy link
Contributor

dvd0101 commented Jan 27, 2015

On Mon, 2015-01-26 at 02:29 -0800, Valerio Maggio wrote:

Per quello che ho intuito, conference è responsabile di tutto il
processo di gestione di talk (schedule e submissions), mentre p3
gestisce gli aspetti della conferenza.
Seguendo questo razionale, ho inteso il campo sub_community più come
"estensione" di (conference.)Talk che come info specifica di PyCon Sei
(e, quindi, p3).
In altre parole, la classe P3Talk l'ho intesa come proxy verso
conference. Tutte le info di un talk sono (e restano, in questo modo)
in conference.talk.
In questo modo si ottiene:

  * una gestione più lineare del processo di salvataggio dei talk
    (risolvendo alcuni problemi di salvataggio e duplicazione
    delle sottomissioni);
  * la info è compilant con i vecchi talk, in quanto il campo è
    empty di default;
  * è possibile abilitare la visualizzazione della info nel
    Back-end di admin con pochissimo effort, senza modificare
    admin-forms e admin-manager

Spero di essermi spiegato ;)

Si ti sei spiegato e hai fatto tutto bene, purtroppo però conference è
condiviso anche da altri siti e contiene solo le informazioni comuni;
estensioni che riguardano siti diversi vengono spostate in modelli fuori
da conference (così come e stato fatto per P3Speaker o
P3ConferenceTicket).

ciao

…es again the sub_community field).

Nevertheless, the `P3Talk` class has been registered to the Admin app along with a new ModelAdmin manager (i.e., `admin.P3TalkAdmin`).
Now it is possibile to filter and manage talks by sub-community in the Admin site, without changing the behaviour of previous `conference` admin.
Other changes includes some minor fixing to the template for talk submission, and updates to the translations.
@leriomaggio
Copy link
Contributor Author

Rollback delle modifiche e nuovo admin di gestione dei P3Talk.
Ora dovrebbe essere tutto compilant con conference. Inoltre, adesso è possibile gestire i P3Talk via Admin filtrando per SubCommunity.

Inoltre, fix di qualche altro bug nella gestione delle form e dei template, e aggiornamento delle traduzioni per i nuovi campi/help_text delle form.

Removed south-generated script (no useless)
Changes to:
- talk.html: now the abstract field has been moved to the top as this is likely the field to be changed. Now this field is shown right after the title, despite the cfp is still open or not!
- dataaccess.py: The coaching mechanism for for p3_talk_data does not work properly and needs to be checked. Therefore, it has been temporarily disabled.
The issue here concerns the fact that changes applied to the abstract of the talk are not updated in the cache and thus not shown in the template (talk.html)
- forms.py: fixed a bug about association of conference.talk and p3talk instances (in case of an already saved talk).
                  applied patch to avoid changing the conference view callback handling talk update after the Cfp is closed.
@leriomaggio
Copy link
Contributor Author

Fixed anche bug di update dei talk (con CfP aperta e chiusa).
Gli errori segnalati da Simone in ML sono dovuti ad un erroneo posizionamento delle URL in pycon/urls.py (che avevo già corretto e dimenticato di segnalare nella descrizone della PR :)

@dvd0101 dvd0101 merged commit 28d4a47 into pythonitalia:develop Feb 10, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants