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

Suggest adding coarse ID in onboarding_species_name #763

Merged

Conversation

reosarevok
Copy link
Contributor

@reosarevok reosarevok commented Dec 21, 2019

With the current message, people probably don't realize that it's ok if they just add a coarse ID, and that might make them leave the field blank rather than "sound ignorant". This changes the string so that it's clear that adding coarse IDs is ok.

Let me know if this is not the right file where to make this change, since I'm not really familiar with the codebase just yet :)

@budowski
Copy link
Contributor

@kueda - what do you think? The problem is that we will also need to update all of the translations as well

@reosarevok
Copy link
Contributor Author

@kueda
Copy link
Member

kueda commented Dec 27, 2019

For what it's worth, I don't think most people read the onboarding text, but if we want to mention the possibility of coarse IDs, how about this slightly shorter version? Name what you saw if you can, even if that's just "insect" or "bird," or just leave this blank. Others may help you identify it!

With the current message, people probably don't realize that it's ok
if they just add a coarse ID, and that might make them leave the field
blank rather than "sound ignorant". This changes the string so that
it's clear that adding coarse IDs is ok.
@reosarevok reosarevok force-pushed the clarify-onboarding_species_name branch from 2aae955 to 2c19374 Compare December 28, 2019 11:32
@reosarevok
Copy link
Contributor Author

Changed this to the string @kueda suggested then :)

@kueda kueda merged commit 06d484c into inaturalist:master Dec 31, 2019
@reosarevok reosarevok deleted the clarify-onboarding_species_name branch December 31, 2019 20:18
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

3 participants