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.

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

Stop using a copy of the talk description on the accepted talks page #91

Open
mw44118 opened this issue Jul 1, 2015 · 2 comments
Open

Comments

@mw44118
Copy link

mw44118 commented Jul 1, 2015

Hi there -- PyOhio uses a fork of symposion, and I just created an issue in our repo, and a friend said I should add it here.

So, I'm copying / pasting this below.

Here's a fun fact: when a talk is accepted, the system copies the talk description.

So, speakers then go back and edit their original proposal, but don't see the changes show up on the accepted talks page but the DO see them on the original proposal for their talk.

Then they ask me to refresh the site, and I never remember how to do that.

The right way for this to work is to only keep a single copy of the data.

Somebody explained that the idea was to prevent people from editing a talk after acceptance, or after voting (I don't remember the specifics), and that's not necessarily a bad idea.

But I would prefer to support that feature explicitly by blocking edits, not by copying data.

I found these notes from last year when I had to deal with this problem:

This is the process I followed to refresh the talk data in the schedule.

Go to the review page for a talk and change the status from Accepted to Standby. This has a side effect of removing the talk from the schedule.
This is the review page for the configman talk:
http://www.pyohio.org/reviews/review/117/

Immediately change the talk back from Standby to Accepted.

Go to the edit schedule page:

http://www.pyohio.org/schedule/talks/edit/

and then find the plus symbol where the talk used to be. Click that
and in the popup modal, select the talk.

It seems to work OK. the sexism talk is now "diversity in tech". I
don't know what exact changes were made in the configman talk though,
so I can't be sure of that one.

@miurahr
Copy link
Contributor

miurahr commented Jul 18, 2015

It may be better to add an admin "action" to bring description from proposal to presentation.

@miurahr
Copy link
Contributor

miurahr commented Jul 23, 2015

In my local development, I've linked proposal and presentation details directly.
With this modification, changes speaker made will be reflected immediately to schedule details.

miurahr@5cc7720#diff-4602d0514027b76396671c88251b11f6R188

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants