Skip to content
This repository has been archived by the owner on Apr 29, 2022. It is now read-only.

WIP first draft of add co-speaker feature #925

Open
wants to merge 1 commit into
base: dev/ep2019
Choose a base branch
from

Conversation

artcz
Copy link
Contributor

@artcz artcz commented May 1, 2019

No description provided.

@umgelurgel
Copy link
Contributor

I'm not sure I understand the workflow; let's assume A creates a talk, B wants to be a co-speaker.

  1. Should adding a co-speaker only be allowed during CfP?
  2. B only sees his talks in user-panel - how should he access the preview proposal page for A's talk? Should the url be sent directly to him? If so, I guess we should add a button to A's preview that allows them to invite someone to become a co-speaker.
  3. Right now, anyone with the reverse('preview_proposal', args=(talk.slug,)) url could in theory add themselves as a co-speaker. How should we permission it? Users could request to be co-speakers on any talk and the speaker would have to approve them, but that would lead to speaker spamming; or we could use the email mechanism from point 2, to ensure that only invited users can become speakers.

Also related to point 2 and cfp - should we disable submit_proposal_step2_add_speakers outside of cfp?

What do you think @artcz?

@umgelurgel umgelurgel changed the title first draft of add co-speaker feature WIP first draft of add co-speaker feature Nov 16, 2019
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.

2 participants