Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow multiple Speakers for a session #251

Open
jandd opened this issue Mar 5, 2020 · 2 comments
Open

Allow multiple Speakers for a session #251

jandd opened this issue Mar 5, 2020 · 2 comments
Assignees
@jandd
Labels
api backend django django-admin enhancement New feature or request frontend Design, HTML, CSS, JS help wanted Extra attention is needed priority medium

Comments

@jandd
Copy link
Contributor

jandd commented Mar 5, 2020

Is your feature request related to a problem? Please describe.
The current data model does only allow a single PublishedSpeaker/Speaker for each Talk instance. There are talks/sessions that are performed by multiple speakers and we should allow this in the data model.

Describe the solution you'd like
Allow adding additional speakers for a talk. The new speaker should have its own speaker data and should be able to edit the data by him/herself.

Additional context
Data model, export scripts, frontend and test data and tests have to be adapted.
@jandd jandd added api backend django django-admin enhancement New feature or request frontend Design, HTML, CSS, JS help wanted Extra attention is needed priority medium labels Mar 5, 2020
@jandd jandd self-assigned this Mar 10, 2020
@jandd
Copy link
Contributor Author

jandd commented Mar 11, 2020

I started work on this feature in a separate feature branch at https://github.com/jandd/devday_website/tree/multi-speaker-support

@jandd jandd mentioned this issue Mar 11, 2020
Merged
jandd added a commit to jandd/devday_website that referenced this issue Mar 12, 2020
@jandd
Fix AttributeError on speaker profile page
0e4d0af 
The change in devdaydresden#255 introduced a regression on the speaker profile page
(AttributeError in prefetch_related) that is fixed by this commit. A
test to reproduce the error is included.

addresses devdaydresden#251
@jandd jandd mentioned this issue Mar 12, 2020
Merged
@jandd
Copy link
Contributor Author

jandd commented May 13, 2022

Multiple speakers are supported in the backend. There is no support to add another speaker for speakers during the CfP period yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jandd jandd

Labels
api backend django django-admin enhancement New feature or request frontend Design, HTML, CSS, JS help wanted Extra attention is needed priority medium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jandd