-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Collection of device's essential information when user want to share feedback #2494
Comments
@nicolas-raoul Should I keep the support information same for all languages? |
@madhurgupta10 Localization would be a good thing, I would say. |
@nicolas-raoul Cool, well in that case shall I just create strings in res file for English and translateWiki would take care of the rest, right? |
Yes, that's it :-)
…On Thu, Feb 21, 2019 at 4:53 PM Madhur Gupta ***@***.***> wrote:
@nicolas-raoul <https://github.com/nicolas-raoul> Cool, well in that case
shall I just create strings in res file for English and translate would
take care of the rest, right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2494 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGFBvxQO1JfCrYvKphvpUd4zHLOd2tnks5vPlCMgaJpZM4bFg7g>
.
|
@nicolas-raoul Done, shall I add anything else? |
Is that how it will appear to English-language users?
If yes, why not the friendlier "API level" instead of "API_LEVEL"? Same for
the others.
Also, since "App version" is in the body, we can remove it from the email
title :-)
Thanks!
…On Thu, Feb 21, 2019 at 6:10 PM Madhur Gupta ***@***.***> wrote:
@nicolas-raoul <https://github.com/nicolas-raoul> Done, shall I add
anything else?
[image: screenshot_20190221-143825]
<https://user-images.githubusercontent.com/30932899/53156965-89133b80-35e6-11e9-9bb3-0607567bc04a.png>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2494 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGFBs-xmxXsJGMmfV9BN2b1uaZm-5ztks5vPmJ4gaJpZM4bFg7g>
.
|
@nicolas-raoul Sure, I will make the changes |
@nicolas-raoul I think keeping the app version in the title would be better than the body, but I can do it, either way, let me know :) |
I let you choose what you think is best :-) |
@nicolas-raoul PR is ready :) |
@nicolas-raoul will localising these strings be useful? Most of the people who are part of the commons and commons private Google groups understand English whereas if these meta deta come in any of the ~50-100 supported languages then it might be difficult to interpret. As these strings would be seen by one who is reading the feedback/logs IMO it would be better not to localise it. My understanding might be wrong. Please correct me if I am missing something. :) |
From the perspective of a non-technical non-English-language person who wants to send feedback about the app, it might be scary to have obscure codes and words in a language you do not understand added to your feedback. For us English-speaking developers the content sounds obvious, but imagine you want to send feedback about something and this text gets attached to your email:
Would you press |
@nicolas-raoul I agree this will be scary, as the person doesn't know what information is attached to the email. What if we replace |
@madhurgupta10 I think |
@nicolas-raoul @maskaravivek Okay 👍 I will make the changes, thanks for the discussion :) |
Summary:
When a user wants to share some feedback, the current scenario does not include the device name, model number, network type, etc.
Automatically adding this to the email template would help the team to address the feedback precisely.
Steps to reproduce:
Open the app < hamburger menu < Feedback
Would you like to work on the issue?
Yes, I want to work on this issue :)
The text was updated successfully, but these errors were encountered: