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

add some new flags to speaker profiles! #628

Merged
merged 6 commits into from Sep 24, 2017
Merged

Conversation

ewdurbin
Copy link
Member

@ewdurbin ewdurbin commented Sep 23, 2017

Hello @jasonamyers, @lorenanicole, @jackiekazil!

I'd like to welcome your feedback on collecting some new booleanish fields during creation of speaker profiles.

Screencap of how this appears as of 5f9b921:

screen shot 2017-09-24 at 11 48 03 am

@lorenanicole
Copy link

lorenanicole commented Sep 23, 2017

@ewdurbin @jasonamyers I'd like to add upon the require financial assistance to indicate that this isn't binding. Some people may not know or their need for financial assistance may change, that could lead to confusion that if that situation changes they'll lose their acceptance or not. Better to suggest in the language we care about them being real about their potential b/c we <3 them! Thoughts? If you think it's worthwhile, I can add some language in?

@jackiekazil
Copy link

jackiekazil commented Sep 23, 2017 via email

@jasonamyers
Copy link

@lorenanicole I think that is a fine point. How about after the Check here to indicate that you require assistance. We add a line that says “We understand situations can change, and we are here for you. Let us know at pycon-aid@python.org

@jasonamyers
Copy link

jasonamyers commented Sep 23, 2017

@jackiekazil They are placeholders for later. (I had that same reaction 😄 ) @ewdurbin should those lines come from us, or some other mentoring group?

@ewdurbin
Copy link
Member Author

@jackiekazil @jasonamyers Ah, yes :) I'm still working on copy for those.

]
help_texts = {
'mobile_number': _(
u"For on-site use only,"
u" in case we need to get in touch with you."
),
'interested_mentee': _(
Copy link
Member Author

@ewdurbin ewdurbin Sep 23, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mariatta any suggestions for good information to put in these help texts?

The tl;dr I'm looking for is "check these boxes and we'll get back to you as soon as we have more information on this program", but I want it to sound more inviting than that" 🙁

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ewdurbin! So exciting that this is happening!

I like the different choices you have here. While running the PyCascades speaker mentorship program, we found some submitters were only interested in mentorship if their proposal is accepted (e.g. wanting feedback about slides). Some others would like more feedback at proposal preparation stage.

I think it's not necessary to mention that this is something "new".

What do you think about "We want to support our speakers. Let us know the different ways we can help you, and will get back once we have more information"

Maybe you can also mention that this is not limited to only new/first time speakers.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mariatta thank you so much for your input, it's been invaluable! do you mind if I request a review of my draft when it is complete?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can follow up with another effort specifically for mentorship around preparation/practice after talks are accepted!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For sure!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright @Mariatta updated copy for mentors/mentees! Any/all feedback welcome :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thanks :)

@ewdurbin
Copy link
Member Author

screen cap as of 5f9b921:

screen shot 2017-09-24 at 11 48 03 am

u"kind of help you're interested in providing and you'll hear "
u"from us when we have more information!"
),
'financial_support': _(
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jackiekazil @lorenanicole @jasonamyers new copy updates for financial support text.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 from me, thanks @ewdurbin!!!!!!!

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I concur with @lorenanicole 👍

@ewdurbin ewdurbin merged commit b84f3e3 into develop Sep 24, 2017
@ewdurbin ewdurbin deleted the speaker_profile_upates branch September 24, 2017 16:26
@ewdurbin
Copy link
Member Author

thanks for participating :)

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

Successfully merging this pull request may close these issues.

None yet

5 participants