Merge lp://staging/~cjohnston/summit/registration-form into lp://staging/summit

Proposed by Chris Johnston
Status: Merged
Approved by: Chris Johnston
Approved revision: 522
Merged at revision: 523
Proposed branch: lp://staging/~cjohnston/summit/registration-form
Merge into: lp://staging/summit
Diff against target: 595 lines (+503/-1)
10 files modified
EXTERNALS (+7/-0)
summit/common/widgets.py (+122/-0)
summit/media/css/style.css (+17/-0)
summit/media/js/events-ui.js (+11/-0)
summit/schedule/forms.py (+31/-0)
summit/schedule/templates/schedule/form.html (+41/-0)
summit/schedule/tests/__init__.py (+1/-0)
summit/schedule/tests/registration.py (+222/-0)
summit/schedule/views.py (+46/-1)
summit/urls.py (+5/-0)
To merge this branch: bzr merge lp://staging/~cjohnston/summit/registration-form
Reviewer Review Type Date Requested Status
Adnane Belmadiaf Approve
Review via email: mp+156978@code.staging.launchpad.net

This proposal supersedes a proposal from 2013-04-03.

Commit message

Adds a registration form to register as attending a sprint.

Description of the change

This adds a registration form as well as a generic form.html template, which I hope to be able to reuse for more (all) forms instead of having to have a custom template for each form. Note: This does not include a link to the registration page. summit.html will need some re-work for that, so I will be doing that in a separate branch.

To post a comment you must log in.
Revision history for this message
Adnane Belmadiaf (daker) wrote :

The timewidget doesn't select the correct value( see bug 1164296 ), for the forms.Select there should an initial value if the user have already set his availability.

519. By Chris Johnston

Use async loading

520. By Chris Johnston

Fixes issue with select widget

521. By Chris Johnston

Update from trunk

522. By Chris Johnston

Removes css/js

Revision history for this message
Adnane Belmadiaf (daker) wrote :

+1 from me :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches